Re: [Xen-devel] [PATCH 20 of 26] xl: free all data on exit from the domain monitor daemon

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [Xen-devel] [PATCH 20 of 26] xl: free all data on 
exit from the domain monitor daemon"):
> I think it needs an out_closelog label which does the closing since the
> regular out label is used from places which do not open the logfile.
> I'll make that change.

You could avoid messing about with different out labels (always
error-prone) by closing logfile iff it has been reopened:
  if (logfile != 2) close(logfile)

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel