[PATCH v2 1/1] windowscodecs: Simplify png_decoder_get_metadata_blocks using realloc.

[Jeff Smith]

From: Jeff Smith

---
 dlls/windowscodecs/libpng.c | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/dlls/windowscodecs/libpng.c b/dlls/windowscodecs/libpng.c
index 137a8e15e33..2c660b83e72 100644
--- a/dlls/windowscodecs/libpng.c
+++ b/dlls/windowscodecs/libpng.c
@@ -377,10 +377,12 @@ static HRESULT CDECL png_decoder_get_metadata_blocks(struct decoder* iface,
     do
     {
         hr = stream_seek(This->stream, seek, STREAM_SEEK_SET, &chunk_start);
-        if (FAILED(hr)) goto end;
+        if (FAILED(hr))
+            break;

hr = read_png_chunk(This->stream, chunk_type, NULL, &chunk_size);
-        if (FAILED(hr)) goto end;
+        if (FAILED(hr))
+            break;

if (chunk_type[0] >= 'a' && chunk_type[0] <= 'z' &&
             memcmp(chunk_type, "tRNS", 4) && memcmp(chunk_type, "pHYs", 4))
@@ -389,23 +391,17 @@ static HRESULT CDECL png_decoder_get_metadata_blocks(struct decoder* iface,
             if (*count == metadata_blocks_size)
             {
                 struct decoder_block *new_metadata_blocks;
-                ULONG new_metadata_blocks_size;

-                new_metadata_blocks_size = 4 + metadata_blocks_size * 2;
-                new_metadata_blocks = malloc(new_metadata_blocks_size * sizeof(*new_metadata_blocks));
+                metadata_blocks_size = 4 + metadata_blocks_size * 2;
+                new_metadata_blocks = realloc(result, metadata_blocks_size * sizeof(*new_metadata_blocks));

if (!new_metadata_blocks)
                 {
                     hr = E_OUTOFMEMORY;
-                    goto end;
+                    break;
                 }

-                memcpy(new_metadata_blocks, result,
-                    *count * sizeof(*new_metadata_blocks));
-
-                free(result);
                 result = new_metadata_blocks;
-                metadata_blocks_size = new_metadata_blocks_size;
             }

result[*count].offset = chunk_start;
@@ -417,7 +413,6 @@ static HRESULT CDECL png_decoder_get_metadata_blocks(struct decoder* iface,
         seek = chunk_start + chunk_size + 12; /* skip data and CRC */
     } while (memcmp(chunk_type, "IEND", 4));

-end:
     if (SUCCEEDED(hr))
     {
         *blocks = result;