From 3c1bdfd5d905eb5e916d588d59494d53ea968eb0 Mon Sep 17 00:00:00 2001
From: Michael Brown <mcb30@ipxe.org>
Date: Sun, 27 Mar 2011 19:36:00 +0100
Subject: [PATCH] [image] Remove redundant call to basename()

There is no need to explicitly call basename() to construct an image
name in imgfetch_core_exec(), since image_set_uri() will do so
automatically anyway (and will do so without getting confused by URIs
with query strings).

Signed-off-by: Michael Brown <mcb30@ipxe.org>
---
 src/hci/commands/image_cmd.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/hci/commands/image_cmd.c b/src/hci/commands/image_cmd.c
index c9a188a2d..1ff3ff49b 100644
--- a/src/hci/commands/image_cmd.c
+++ b/src/hci/commands/image_cmd.c
@@ -22,7 +22,6 @@ FILE_LICENCE ( GPL2_OR_LATER );
 #include <stdlib.h>
 #include <stdio.h>
 #include <errno.h>
-#include <libgen.h>
 #include <getopt.h>
 #include <ipxe/image.h>
 #include <ipxe/command.h>
@@ -76,8 +75,6 @@ static int imgfetch_core_exec ( int argc, char **argv,
 
 	/* Parse URI string */
 	uri_string = argv[optind];
-	if ( ! opts.name )
-		opts.name = basename ( uri_string );
 
 	/* Parse command line */
 	if ( argv[ optind + 1 ] != NULL ) {