[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [RFC PATCH 2/6] libxl: stop using libxl__xs_mkdir() for ~/control/shutdown
> -----Original Message----- > From: Ian Campbell [mailto:ian.campbell@xxxxxxxxxx] > Sent: 16 November 2015 13:43 > To: Paul Durrant; xen-devel@xxxxxxxxxxxxxxxxxxxx > Cc: Ian Jackson; Stefano Stabellini; Wei Liu > Subject: Re: [RFC PATCH 2/6] libxl: stop using libxl__xs_mkdir() for > ~/control/shutdown > > On Mon, 2015-11-16 at 13:16 +0000, Paul Durrant wrote: > > > > > > > +ÂÂÂÂret = vasprintf(&s, fmt, ap); > > > > ÂÂÂÂÂif (ret == -1) { > > > > ÂÂÂÂÂÂÂÂÂreturn -1; > > > > ÂÂÂÂÂ} > > > > ÂÂÂÂÂxs_write(ctx->xsh, t, path, s, ret); > > > > +ÂÂÂÂif (perms) > > > > +ÂÂÂÂÂÂÂÂxs_set_permissions(ctx->xsh, t, path, perms, num_perms); > > > > > > This can fail, can't it? (OTOH so can xs_write, so maybe there is some > > > reason we apparently don't care for such things here?) > > > > > > > That's a good point. I would have thought wiring an xs_write() failure > > back through to the caller would be a good idea (and same goes for > > xs_set_permissions, I guess). > > git tells me it has been this way since libxl was first committed to the > repo. I can't think of a good reason to squash these errors. > I'll split out this patch and the name change one and send a small cleanup series. I'll wait for Ian's docs review before sending the rest. Paul > Ian. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |