From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: [PATCH v4 4/5] xl: Return an error on failed cd-insert Date: Fri, 18 Dec 2015 12:23:44 +0000 Message-ID: <1450441425-10755-5-git-send-email-george.dunlap@eu.citrix.com> References: <1450441425-10755-1-git-send-email-george.dunlap@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1450441425-10755-1-git-send-email-george.dunlap@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org Cc: George Dunlap , Ian Jackson , Wei Liu , Ian Campbell List-Id: xen-devel@lists.xenproject.org This makes xl more useful in scripts. The strange thing about this is that the internal cd_insert function *already* returned something appropriate, and cd-eject was using it, but cd-insert wasn't. Also: * Rework cd_insert to return EXIT_FAILURE and EXIT_SUCCESS rather than magic constants * Use 'r' for non-libxl return code, as specified in CODING_STYLE Signed-off-by: George Dunlap --- v4: - Use EXIT_{SUCCESS,FAILURE} rather than magic constants in cd_insert - Use 'r' rather than 'rc' for non-libxl return code - Fixed typo in changelog CC: Ian Campbell CC: Ian Jackson CC: Wei Liu --- tools/libxl/xl_cmdimpl.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 1a8b4a1..0c3756b 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -3336,7 +3336,7 @@ static int cd_insert(uint32_t domid, const char *virtdev, char *phys) char *buf = NULL; XLU_Config *config = 0; struct stat b; - int rc = 0; + int r; xasprintf(&buf, "vdev=%s,access=r,devtype=cdrom,target=%s", virtdev, phys ? phys : ""); @@ -3352,18 +3352,22 @@ static int cd_insert(uint32_t domid, const char *virtdev, char *phys) && stat(disk.pdev_path, &b)) { fprintf(stderr, "Cannot stat file: %s\n", disk.pdev_path); - rc = 1; + r = EXIT_FAILURE; + goto out; + } + + if (libxl_cdrom_insert(ctx, domid, &disk, NULL)) { + r = EXIT_FAILURE; goto out; } - if (libxl_cdrom_insert(ctx, domid, &disk, NULL)) - rc=1; + r = EXIT_SUCCESS; out: libxl_device_disk_dispose(&disk); free(buf); - return rc; + return r; } int main_cd_eject(int argc, char **argv) @@ -3397,8 +3401,7 @@ int main_cd_insert(int argc, char **argv) virtdev = argv[optind + 1]; file = argv[optind + 2]; - cd_insert(domid, virtdev, file); - return 0; + return cd_insert(domid, virtdev, file); } int main_console(int argc, char **argv) -- 2.1.4