> On May 26, 2017, at 9:14 PM, Ben Pfaff <[email protected]> wrote: > > This fixes a seemingly severe memory leak in ipfix_send_template_msgs(). > This function was setting up a buffer with a stub, but only the first 4 > or 8 bytes of the stub were actually used because the "sizeof" call used > to size it was actually getting the size of a pointer. It never freed > the buffer, leaking it. > > Additionally, after this code sent a template message, it started over > from the same undersized stub, leaking another block of memory. > > This commit fixes both problems. > > Found by Coverity. > > CC: Romain Lenglet <[email protected]> > Reported-at: > https://scan3.coverity.com/reports.htm#v16889/p10449/fileInstanceId=14762995&defectInstanceId=4304799&mergedDefectId=180398 > Signed-off-by: Ben Pfaff <[email protected]>
Acked-by: Justin Pettit <[email protected]> Thanks for running through the Coverity reports and fixing all these issues! --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
