Re: [libvirt] [PATCH] Fix openlog() ident usage
On Tue, Jan 20, 2009 at 08:36:27PM +0100, Daniel Veillard wrote:
On Tue, Jan 20, 2009 at 11:14:03AM -0800, john.levon(a)sun.com wrote:
> It cannot be freed, as it's used by reference in the syslog code, at
> least on Solaris.
The requirement is apparently not present on Linux, sorry :-)
I would prefer to keep the reference around and not leak it for
example on reloads
Another version with an extra strdup but making clearer what
the change do and fixing the closelog/free order,
if tested positively please apply, thanks !
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
Attachments:
- ident.patch (text/plain — 1012 bytes)