From 0db1137dbf5008d456b9cc80a5cf1222467fed9d Mon Sep 17 00:00:00 2001 From: ths Date: Tue, 20 Feb 2007 00:12:07 +0000 Subject: [PATCH] Change -tftp option to take a root directory, by Anthony Liguori. git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@2441 c046a42c-6fe2-441c-8c8c-71466251a162 --- qemu-doc.texi | 11 +++++------ slirp/tftp.c | 12 +++++++++--- vl.c | 2 +- 3 files changed, 15 insertions(+), 10 deletions(-) diff --git a/qemu-doc.texi b/qemu-doc.texi index b23b033..fe69e80 100644 --- a/qemu-doc.texi +++ b/qemu-doc.texi @@ -444,13 +444,12 @@ Indicate that no network devices should be configured. It is used to override the default configuration (@option{-net nic -net user}) which is activated if no @option{-net} options are provided. -@item -tftp prefix +@item -tftp dir When using the user mode network stack, activate a built-in TFTP -server. All filenames beginning with @var{prefix} can be downloaded -from the host to the guest using a TFTP client. The TFTP client on the -guest must be configured in binary mode (use the command @code{bin} of -the Unix TFTP client). The host IP address on the guest is as usual -10.0.2.2. +server. The files in @var{dir} will be exposed as the root of a TFTP server. +The TFTP client on the guest must be configured in binary mode (use the command +@code{bin} of the Unix TFTP client). The host IP address on the guest is as +usual 10.0.2.2. @item -bootp file When using the user mode network stack, broadcast @var{file} as the BOOTP diff --git a/slirp/tftp.c b/slirp/tftp.c index b3947eb..96d00e6 100644 --- a/slirp/tftp.c +++ b/slirp/tftp.c @@ -102,8 +102,15 @@ static int tftp_read_data(struct tftp_session *spt, u_int16_t block_nr, { int fd; int bytes_read = 0; + char buffer[1024]; + int n; - fd = open(spt->filename, O_RDONLY | O_BINARY); + n = snprintf(buffer, sizeof(buffer), "%s/%s", + tftp_prefix, spt->filename); + if (n >= sizeof(buffer)) + return -1; + + fd = open(buffer, O_RDONLY | O_BINARY); if (fd < 0) { return -1; @@ -325,8 +332,7 @@ static void tftp_handle_rrq(struct tftp_t *tp, int pktlen) /* only allow exported prefixes */ - if (!tftp_prefix - || (strncmp(spt->filename, tftp_prefix, strlen(tftp_prefix)) != 0)) { + if (!tftp_prefix) { tftp_send_error(spt, 2, "Access violation", tp); return; } diff --git a/vl.c b/vl.c index fd944e0..7a4058f 100644 --- a/vl.c +++ b/vl.c @@ -6402,7 +6402,7 @@ void help(void) " is provided, the default is '-net nic -net user'\n" "\n" #ifdef CONFIG_SLIRP - "-tftp prefix allow tftp access to files starting with prefix [-net user]\n" + "-tftp dir allow tftp access to files in dir [-net user]\n" "-bootp file advertise file in BOOTP replies\n" #ifndef _WIN32 "-smb dir allow SMB access to files in 'dir' [-net user]\n" -- 1.7.9.5