Browse Source

[autoboot] Retain initial-slash (if present) when constructing TFTP URIs

When we boot from a DHCP-supplied filename, we previously relied on
the fact that the current working URI is set to tftp://[next-server]/
in order to resolve the filename into a full tftp:// URI.  However,
this process will eliminate the distinction between filenames with and
without initial slashes:

 cwuri="tftp://10.0.0.1/" filename="vmlinuz"  => URI="tftp://10.0.0.1/vmlinuz"
 cwuri="tftp://10.0.0.1/" filename="/vmlinuz" => URI="tftp://10.0.0.1/vmlinuz"

This distinction is important for some TFTP servers.  We now
explicitly construct a string of the form

 "tftp://[next-server]/filename"

so that a filename with an initial slash will result in a URI
containing a double-slash, e.g.

 "tftp://10.0.0.1//vmlinuz"

The TFTP code always strips a single initial slash, and so ends up
presenting the correct path to the server.

URIs entered explicitly by users at the command line must include a
double slash if they want an initial slash presented to the TFTP
server:

  "kernel tftp://10.0.0.1/vmlinuz"  => filename="vmlinuz"
  "kernel tftp://10.0.0.1//vmlinuz" => filename="/vmlinuz"
tags/v0.9.4
Michael Brown 16 years ago
parent
commit
481a21798d
2 changed files with 31 additions and 3 deletions
  1. 1
    0
      src/include/gpxe/settings.h
  2. 30
    3
      src/usr/autoboot.c

+ 1
- 0
src/include/gpxe/settings.h View File

215
 extern struct setting priority_setting __setting;
215
 extern struct setting priority_setting __setting;
216
 extern struct setting bios_drive_setting __setting;
216
 extern struct setting bios_drive_setting __setting;
217
 extern struct setting uuid_setting __setting;
217
 extern struct setting uuid_setting __setting;
218
+extern struct setting next_server_setting __setting;
218
 
219
 
219
 /**
220
 /**
220
  * Initialise a settings block
221
  * Initialise a settings block

+ 30
- 3
src/usr/autoboot.c View File

24
 #include <gpxe/settings.h>
24
 #include <gpxe/settings.h>
25
 #include <gpxe/image.h>
25
 #include <gpxe/image.h>
26
 #include <gpxe/embedded.h>
26
 #include <gpxe/embedded.h>
27
+#include <gpxe/uri.h>
27
 #include <usr/ifmgmt.h>
28
 #include <usr/ifmgmt.h>
28
 #include <usr/route.h>
29
 #include <usr/route.h>
29
 #include <usr/dhcpmgmt.h>
30
 #include <usr/dhcpmgmt.h>
78
 }
79
 }
79
 
80
 
80
 /**
81
 /**
81
- * Boot using filename
82
+ * Boot using next-server and filename
82
  *
83
  *
83
  * @v filename		Boot filename
84
  * @v filename		Boot filename
84
  * @ret rc		Return status code
85
  * @ret rc		Return status code
85
  */
86
  */
86
-static int boot_filename ( const char *filename ) {
87
+static int boot_next_server_and_filename ( struct in_addr next_server,
88
+					   const char *filename ) {
89
+	struct uri *uri;
87
 	struct image *image;
90
 	struct image *image;
91
+	char buf[ 23 /* tftp://xxx.xxx.xxx.xxx/ */ + strlen(filename) + 1 ];
92
+	int filename_is_absolute;
88
 	int rc;
93
 	int rc;
89
 
94
 
95
+	/* Construct URI */
96
+	uri = parse_uri ( filename );
97
+	if ( ! uri ) {
98
+		printf ( "Out of memory\n" );
99
+		return -ENOMEM;
100
+	}
101
+	filename_is_absolute = uri_is_absolute ( uri );
102
+	uri_put ( uri );
103
+	if ( ! filename_is_absolute ) {
104
+		/* Construct a tftp:// URI for the filename.  We can't
105
+		 * just rely on the current working URI, because the
106
+		 * relative URI resolution will remove the distinction
107
+		 * between filenames with and without initial slashes,
108
+		 * which is significant for TFTP.
109
+		 */
110
+		snprintf ( buf, sizeof ( buf ), "tftp://%s/%s",
111
+			   inet_ntoa ( next_server ), filename );
112
+		filename = buf;
113
+	}
114
+
90
 	image = alloc_image();
115
 	image = alloc_image();
91
 	if ( ! image ) {
116
 	if ( ! image ) {
92
 		printf ( "Out of memory\n" );
117
 		printf ( "Out of memory\n" );
135
  */
160
  */
136
 static int netboot ( struct net_device *netdev ) {
161
 static int netboot ( struct net_device *netdev ) {
137
 	char buf[256];
162
 	char buf[256];
163
+	struct in_addr next_server;
138
 	int rc;
164
 	int rc;
139
 
165
 
140
 	/* Open device and display device status */
166
 	/* Open device and display device status */
161
 		return rc;
187
 		return rc;
162
 
188
 
163
 	/* Try to download and boot whatever we are given as a filename */
189
 	/* Try to download and boot whatever we are given as a filename */
190
+	fetch_ipv4_setting ( NULL, &next_server_setting, &next_server );
164
 	fetch_string_setting ( NULL, &filename_setting, buf, sizeof ( buf ) );
191
 	fetch_string_setting ( NULL, &filename_setting, buf, sizeof ( buf ) );
165
 	if ( buf[0] ) {
192
 	if ( buf[0] ) {
166
 		printf ( "Booting from filename \"%s\"\n", buf );
193
 		printf ( "Booting from filename \"%s\"\n", buf );
167
-		return boot_filename ( buf );
194
+		return boot_next_server_and_filename ( next_server, buf );
168
 	}
195
 	}
169
 	
196
 	
170
 	/* No filename; try the root path */
197
 	/* No filename; try the root path */

Loading…
Cancel
Save