Double commit on this one. Overlooked the attached patch, took that as poke for accepted review. 2nd commit reverts the previous patch in favour of the mini_leaks.diff with cleaner naming and one more getpass ifdef fix.
----- Original Message ----- From: "Zdenek Styblik" <zdenek.styb...@gmail.com> To: "ipmitool-devel" <ipmitool-devel@lists.sourceforge.net> Cc: "Ales Ledvinka" <aledv...@redhat.com> Sent: Thursday, January 10, 2013 9:41:37 PM Subject: Code Review - ID: 3595188 - minor leaks via options Hello, ipmitool leaks memory if certain options are passed more than once. Attached is a patch to mend it. Are we ok to commit? Z. ------------------------------------------------------------------------------ Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and much more. Get web development skills now with LearnDevNow - 350+ hours of step-by-step video tutorials by Microsoft MVPs and experts. SALE $99.99 this month only -- learn more at: http://p.sf.net/sfu/learnmore_122812 _______________________________________________ Ipmitool-devel mailing list Ipmitool-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ipmitool-devel
