[downloader] Treat redirection failures as fatal

Debugged-by: Robin Smidsrød <robin@smidsrod.no>
Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/core/downloader.c b/src/core/downloader.c
index ba678f868a92ee8b8da211f03b1d278eda30598e..35b5b0ac69f0e9c49ab57d20abfc2358404ac916 100644 (file)
--- a/src/core/downloader.c
+++ b/src/core/downloader.c
@@ -190,14 +190,18 @@ static int downloader_vredirect ( struct downloader *downloader, int type,
 
                /* Set image URI */
                if ( ( rc = image_set_uri ( downloader->image, uri ) ) != 0 )
-                       return rc;
+                       goto err;
        }
 
        /* Redirect to new location */
        if ( ( rc = xfer_vreopen ( &downloader->xfer, type, args ) ) != 0 )
-               return rc;
+               goto err;
 
        return 0;
+
+ err:
+       downloader_finished ( downloader, rc );
+       return rc;
 }
 
 /** Downloader data transfer interface operations */