Codebase list slapi-nis / 41b540b
Coverity#11937: use proper structure to pass to map_data_set_entry() map_data_set_entry() passes pointers to the lengths of the key and the value to map_data_save_list() which interpretes them as arrays of integers. Alexander Bokovoy authored 10 years ago Nalin Dahyabhai committed 10 years ago
1 changed file(s) with 8 addition(s) and 6 deletion(s). Raw diff Collapse all Expand all
409409 const char *hexchars = "0123456789ABCDEF";
410410 char *rdn, *ndn, *ldif, *plugin_id, *keys[2], *values[2], **ava, *p, *q;
411411 char *usn, *attr, *val;
412 unsigned int rdn_len, value_len, *ava_lens;
412 unsigned int rdn_len[2], value_len[2], *ava_lens;
413413 const char *rdnstr;
414414 int len, i, j, k, count;
415415 Slapi_Entry *entry;
438438 &data->common.inref_attrs,
439439 &data->common.ref_attr_list,
440440 &data->common.inref_attr_list,
441 &rdn_len);
441 rdn_len);
442442 if ((rdn == NULL) || (strlen(rdn) == 0) || (strchr(rdn, '=') == NULL)) {
443443 slapi_log_error(SLAPI_LOG_FATAL, plugin_id,
444444 "no RDN for %s, unsetting domain/map/id "
620620 rdn, ndn, slapi_entry_get_ndn(entry));
621621 keys[0] = (char *) rdnstr;
622622 keys[1] = NULL;
623 rdn_len = strlen(rdnstr);
623 rdn_len[0] = strlen(rdnstr);
624 rdn_len[1] = -1;
624625 values[0] = (char *) slapi_entry_get_ndn(entry);
625626 values[1] = NULL;
626 value_len = -1;
627 value_len[0] = -1;
628 value_len[1] = -1;
627629 map_data_set_entry(data->common.state,
628630 data->common.group, data->common.set, ndn,
629 &rdn_len, keys,
630 &value_len, values,
631 rdn_len, keys,
632 value_len, values,
631633 backend_entry_make_entry_data(source, e_dn,
632634 entry),
633635 backend_entry_free_entry_data);