libsepol resource leak

(I am sending out four patches to libsepol. Another 6 or so which I investigated were not actual bugs. This leaves me with a little over half to still look at, hopefully tomorrow. After that I'll hop to libsemanage or the kernel code).

This patch fixes a bug found by Coverity:

On error in sepol_context_to_string(), 'str' is not freed.

Index: src/context_record.c
===================================================================
@@ -253,5 +253,7 @@ char* sepol_context_to_string(sepol_cont
 
 	err:
 	DEBUG(__FUNCTION__, "could not convert context to string\n");
+	if (str)
+		free(str);
 	return NULL;
 }

--
This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to [email protected] with the words "unsubscribe selinux" without quotes as the message.

Thanks, merged. Did you look at policycoreutils?

Stephen Smalley
National Security Agency


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to [email protected] with
the words "unsubscribe selinux" without quotes as the message.

No, I haven't looked at that yet. Right now I'm sorting through a huge mess of false positives in libsepol...

-serge

--
This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to [email protected] with the words "unsubscribe selinux" without quotes as the message.