Fix a memory leak in ses_get_elm_devnames().
After r307132 the sbuf buffer is malloc()ed, but corresponding sbuf_delete() call was missing. Fix a nearby whitespace bug. MFC after: 3 days Sponsored by: Dell EMC Isilon
This commit is contained in:
parent
58deaaf128
commit
b3db6c0140
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=320372
@ -2684,10 +2684,11 @@ ses_get_elm_devnames(enc_softc_t *enc, encioc_elm_devnames_t *elmdn)
|
|||||||
cam_periph_unlock(enc->periph);
|
cam_periph_unlock(enc->periph);
|
||||||
sbuf_new(&sb, NULL, len, SBUF_FIXEDLEN);
|
sbuf_new(&sb, NULL, len, SBUF_FIXEDLEN);
|
||||||
ses_paths_iter(enc, &enc->enc_cache.elm_map[elmdn->elm_idx],
|
ses_paths_iter(enc, &enc->enc_cache.elm_map[elmdn->elm_idx],
|
||||||
ses_elmdevname_callback, &sb);
|
ses_elmdevname_callback, &sb);
|
||||||
sbuf_finish(&sb);
|
sbuf_finish(&sb);
|
||||||
elmdn->elm_names_len = sbuf_len(&sb);
|
elmdn->elm_names_len = sbuf_len(&sb);
|
||||||
copyout(sbuf_data(&sb), elmdn->elm_devnames, elmdn->elm_names_len + 1);
|
copyout(sbuf_data(&sb), elmdn->elm_devnames, elmdn->elm_names_len + 1);
|
||||||
|
sbuf_delete(&sb);
|
||||||
cam_periph_lock(enc->periph);
|
cam_periph_lock(enc->periph);
|
||||||
return (elmdn->elm_names_len > 0 ? 0 : ENODEV);
|
return (elmdn->elm_names_len > 0 ? 0 : ENODEV);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user