Re: [PATCH 1/1] libctrl: only free() msgb if it was alloc()ed

11 Oct 2011


      On Tue, Oct 11, 2011 at 11:07:35PM +0200, Holger Hans Peter Freyther wrote:
...
On 10/11/2011 10:25 PM, Alexander Huemer wrote:
...
struct ipaccess_head_ext *iph_ext;
-	struct msgb *msg;
+	struct msgb *msg = NULL;
yes
con);
...
-	msgb_free(msg);
+	if (msg)
+		msgb_free(msg);
no, i think we should assume that msgb_free(NULL) will work.
thanks
You are right. I'll resend the patch.