scanner: Only strdup interface_name after making sure it's not NULL

This commit is contained in:
Kristian Høgsberg 2011-11-18 21:23:33 -05:00
parent cf04b0a18f
commit 44186b522f

View file

@ -231,10 +231,8 @@ start_element(void *data, const char *element_name, const char **atts)
arg->type = FD;
else if (strcmp(type, "new_id") == 0) {
arg->type = NEW_ID;
arg->interface_name = strdup(interface_name);
} else if (strcmp(type, "object") == 0) {
arg->type = OBJECT;
arg->interface_name = strdup(interface_name);
} else {
fail(ctx, "unknown type");
}
@ -244,6 +242,7 @@ start_element(void *data, const char *element_name, const char **atts)
case OBJECT:
if (interface_name == NULL)
fail(ctx, "no interface name given");
arg->interface_name = strdup(interface_name);
break;
default:
if (interface_name != NULL)