-
Notifications
You must be signed in to change notification settings - Fork 34
vararg parsing does not properly use the realloced pointer #169
Copy link
Copy link
Open
Description
Bug:
VARARG_TO_ARGV either mallocs or reallocs a pointer to the argv, but the realloc does not use the new pointer.
Spindle/src/utils/handle_vararg.h
Lines 40 to 47 in 585dacf
| else { \ | |
| newp = (char **) spindle_realloc(argv, sizeof(char *) * size); \ | |
| if (!newp) { \ | |
| spindle_free(argv); \ | |
| errno = ENOMEM; \ | |
| return -1; \ | |
| } \ | |
| } \ |
Fix:
else { \
newp = (char **) spindle_realloc(argv, sizeof(char *) * size); \
if (!newp) { \
spindle_free(argv); \
errno = ENOMEM; \
return -1; \
} \
argv = newp; \
} \
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels