On Thu, Mar 19, 2015 at 06:40:30AM +0000, PRAMOD DEVENDRA wrote: > From: Pramod Devendra <pramod.deven...@citrix.com> > > Signed-off-by: Pramod Devendra <pramod.deven...@citrix.com> > CC: Ian Jackson <ian.jack...@eu.citrix.com> > CC: Stefano Stabellini <stefano.stabell...@eu.citrix.com> > CC: Ian Campbell <ian.campb...@citrix.com> > CC: Wei Liu <wei.l...@citrix.com> > --- > Changed since v1: > 1. Make sure sun_path does not overflow. > 2. Close qmp_fd on error.
These lines should be in the commit log itself. Putting them here makes them get discarded when committing. You also need to write about your third change, which was requested by Ian. That is 3. Use "goto out" error handling idiom. Wei. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
