[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv1] slirp: Use strcasecmp() to check tftp mode, t
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCHv1] slirp: Use strcasecmp() to check tftp mode, tsize |
Date: |
Thu, 13 Jan 2011 11:40:21 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Wed, Jan 12, 2011 at 03:57:18PM +0200, Sergei Gavrikov wrote:
> From: Sergei Gavrikov <address@hidden>
>
> According to RFC 1350 (TFTP Revision 2) the mode field can contain any
> combination of upper and lower case; also RFC 2349 propagates that the
> transfer size option ("tsize") is case in-sensitive too.
>
> Current implementation of embedded TFTP server missed that what does
> mess some TFTP clients. Fixed by using STRCASECMP(3) in the required
> places.
>
> Signed-off-by: Sergei Gavrikov <address@hidden>
> Reviewed-by: Stefan Hajnoczi <address@hidden>
I've applied this, thanks both of you.
Cheers
> ---
> slirp/tftp.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/slirp/tftp.c b/slirp/tftp.c
> index 55e4692..1821648 100644
> --- a/slirp/tftp.c
> +++ b/slirp/tftp.c
> @@ -311,7 +311,7 @@ static void tftp_handle_rrq(Slirp *slirp, struct tftp_t
> *tp, int pktlen)
> return;
> }
>
> - if (memcmp(&tp->x.tp_buf[k], "octet\0", 6) != 0) {
> + if (strcasecmp((const char *)&tp->x.tp_buf[k], "octet") != 0) {
> tftp_send_error(spt, 4, "Unsupported transfer mode", tp);
> return;
> }
> @@ -351,7 +351,7 @@ static void tftp_handle_rrq(Slirp *slirp, struct tftp_t
> *tp, int pktlen)
> value = (const char *)&tp->x.tp_buf[k];
> k += strlen(value) + 1;
>
> - if (strcmp(key, "tsize") == 0) {
> + if (strcasecmp(key, "tsize") == 0) {
> int tsize = atoi(value);
> struct stat stat_p;
>
>
- [Qemu-devel] [PATCH] slirp/tftp.c: fix mode field, Sergei Gavrikov, 2011/01/10
- [Qemu-devel] Re: [PATCH] slirp/tftp.c: fix mode field, Sergei Gavrikov, 2011/01/12
- Re: [Qemu-devel] Re: [PATCH] slirp/tftp.c: fix mode field, Stefan Hajnoczi, 2011/01/12
- Re: [Qemu-devel] Re: [PATCH] slirp/tftp.c: fix mode field, Sergei Gavrikov, 2011/01/12
- Re: [Qemu-devel] Re: [PATCH] slirp/tftp.c: fix mode field, Stefan Hajnoczi, 2011/01/12
- Re: [Qemu-devel] Re: [PATCH] slirp/tftp.c: fix mode field, Sergei Gavrikov, 2011/01/12
- [Qemu-devel] [PATCHv1] slirp: Use strcasecmp() to check tftp mode, tsize, Sergei Gavrikov, 2011/01/12
- Re: [Qemu-devel] [PATCHv1] slirp: Use strcasecmp() to check tftp mode, tsize,
Edgar E. Iglesias <=