[pacman-dev] [PATCH] add_fileconflict, don't free if CALLOC fail
If CALLOC fails, conflict is NULL, so we shouldn't
attempt to free it's members, since that would
lead to a NULL pointer dereference.
Signed-off-by: Rikard Falkeborn
2015-10-24 21:51 GMT+02:00 Rikard Falkeborn
If CALLOC fails, conflict is NULL, so we shouldn't attempt to free it's members, since that would lead to a NULL pointer dereference.
Signed-off-by: Rikard Falkeborn
--- Is there a reason none of the alpm_*_free() functions check if the input is NULL? That would be more robust, and what I think would be expected, since free() accepts NULL as input.
I sent at patch to check for NULL in alpm_fileconflict_free and a few more functions. If that's applied, this patch doesn't really add any value.
participants (1)
-
Rikard Falkeborn