[CRIU] Re: [PATCH 5/6] parasite: Don't assign error number on logfd
Cyrill Gorcunov
gorcunov at openvz.org
Tue Feb 14 09:43:00 EST 2012
On Tue, Feb 14, 2012 at 06:20:07PM +0400, Pavel Emelyanov wrote:
> On 02/14/2012 05:42 PM, Cyrill Gorcunov wrote:
> > In case if recv_fd failed we must not assign error
> > number as logfd, it's bloody wrong. Moreover, never
> > close descriptors which were not ever used or point
> > to standart descriptors.
>
> Assigning some other task's 2nd file descriptor is wrong
> too. It's better to keep it -1 to have NO output at all.
>
Not at all. How pray tell you will be debuggin stuff when
someone send you bug-report with no output from parasite?
Having NO output is a way worse than having SOME output.
Cyrill
More information about the CRIU
mailing list