Re: [Xen-devel] [PATCH 6/6] libxl: vncviewer: make autopass work properly

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Thu, 3 Feb 2011, Ian Jackson wrote:
> Stefano Stabellini writes ("Re: [Xen-devel] [PATCH 6/6] libxl: vncviewer: 
> make autopass work properly"):
> > On Tue, 1 Feb 2011, Ian Jackson wrote:
> > > +        if ( autopass_fd < 0 ) {
> > > +            LIBXL__LOG_ERRNO(ctx, LIBXL__LOG_ERROR,
> > > +                             "mkstemp %s failed", tmpname);
> > > +            exit(-1);
> > > +        }
> > 
> > I don't think we should call exit here, this is a library not an
> > executable. However instead of just skipping autopass we should avoid
> > exec'ing vncviewer completely and return and error.
> 
> The caller must already tolerate the function simply causing the
> process to die, because execve can fail like that.  Ie,
> libxl_exec_vncviewer is already called only after fork.  So it seemed
> best to have the function always fail the same way.
> 
> If we had a function which merely returned the vnc connection info, I
> would agree with you that it shouldn't exit.  We should do that in 4.2.

Even though it might be tolerable to have this function exits, I still
don't see any benefits as opposed to failing with an error, considering
that this function returns an integer and the long term plan would be
to return an error anyway.
It is just a matter of checking the return value in
tools/libxl/xl_cmdimpl.c:vncviewer.

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