Coverity is concerned that if buffer is NULL, strtok_r() is called with
two NULL arguments, causing a NULL pointer dereference.
CID: 37968
status = powerdns_get_data(item, &buffer, &buffer_size);
if (status != 0)
return (-1);
status = powerdns_get_data(item, &buffer, &buffer_size);
if (status != 0)
return (-1);
+ if ((buffer == NULL) || (buffer_size == 0)) {
+ return EINVAL;
+ }
if (item->fields_num != 0) {
fields = (const char *const *)item->fields;
if (item->fields_num != 0) {
fields = (const char *const *)item->fields;