* PATCH: fix memory leak in setfiles.c
@ 2006-10-09 17:28 Todd Miller
2006-10-11 6:38 ` Joshua Brindle
0 siblings, 1 reply; 2+ messages in thread
From: Todd Miller @ 2006-10-09 17:28 UTC (permalink / raw)
To: SE Linux
[-- Attachment #1: Type: text/plain, Size: 53 bytes --]
Add a missing freecon() on an error path.
- todd
[-- Attachment #2: setfiles.c.diff --]
[-- Type: application/octet-stream, Size: 259 bytes --]
--- setfiles.c.DIST 2006-10-09 13:20:10.000000000 -0400
+++ setfiles.c 2006-10-09 13:23:25.000000000 -0400
@@ -357,6 +357,7 @@
"%s: %s not reset customized by admin to %s\n",
progname, my_file, context);
}
+ freecon(context);
goto out;
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: PATCH: fix memory leak in setfiles.c
2006-10-09 17:28 PATCH: fix memory leak in setfiles.c Todd Miller
@ 2006-10-11 6:38 ` Joshua Brindle
0 siblings, 0 replies; 2+ messages in thread
From: Joshua Brindle @ 2006-10-11 6:38 UTC (permalink / raw)
To: Todd Miller; +Cc: SE Linux
Todd Miller wrote:
> Add a missing freecon() on an error path.
>
> - todd
>
Thanks for finding and fixing this memory leak. For future reference,
inlined patches that apply with -p1 from the top of the repository are
preferred.
Acked-By: Joshua Brindle <jbrindle@tresys.com>
--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-10-11 6:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-09 17:28 PATCH: fix memory leak in setfiles.c Todd Miller
2006-10-11 6:38 ` Joshua Brindle
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.