small fix for ccan/take/take.c

[ccan] / ccan / take / take.c

diff --git a/ccan/take/take.c b/ccan/take/take.c
index c628aac0dc97ff663db216e4f05a14b8cfd38bbf..4833bf93576473a964e0cb6eab120af7f49c1fd5 100644 (file)
--- a/ccan/take/take.c
+++ b/ccan/take/take.c
@@ -32,9 +32,20 @@ void *take_(const void *p, const char *label)
                }
                takenarr = new;
                /* Once labelarr is set, we maintain it. */
-               if (labelarr)
-                       labelarr = realloc(labelarr,
-                                          sizeof(*labelarr) * (max_taken+1));
+               if (labelarr) {
+                        const char **labelarr_new;
+                       labelarr_new = realloc(labelarr,
+                                              sizeof(*labelarr) * (max_taken+1));
+                        if (labelarr_new) {
+                                labelarr = labelarr_new;
+                        } else {
+                                /* num_taken will be out of sync with the size of
+                                 * labelarr after realloc failure.
+                                 * Just pretend that we never had labelarr allocated. */
+                                free(labelarr);
+                                labelarr = NULL;
+                        }
+                }
                max_taken++;
        }
        if (unlikely(labelarr))