From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Warren <biggerbadderben@gmail.com>
Date: Mon, 10 Aug 2009 14:54:28 -0700
Subject: [U-Boot] [PATCH] net/tftp.c: fix warning: pointer targets
	differ in signedness
In-Reply-To: <1249891294-7647-1-git-send-email-wd@denx.de>
References: <1249891294-7647-1-git-send-email-wd@denx.de>
Message-ID: <4A809714.6040208@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Wolfgang,

Wolfgang Denk wrote:
> tftp.c:294: warning: pointer targets in passing argument 1 of 'strlen'
> differ in signedness
>
> This was only visible for the utx8245 board which seems to have DEBUG
> enabled.
>
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> ---
>  net/tftp.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/net/tftp.c b/net/tftp.c
> index 74d9e42..fb98a34 100644
> --- a/net/tftp.c
> +++ b/net/tftp.c
> @@ -291,7 +291,9 @@ TftpHandler (uchar * pkt, unsigned dest, unsigned src, unsigned len)
>  		break;
>  
>  	case TFTP_OACK:
> -		debug("Got OACK: %s %s\n", pkt, pkt+strlen(pkt)+1);
> +		debug("Got OACK: %s %s\n",
> +			pkt,
> +			pkt + strlen((char *)pkt) + 1);
>  		TftpState = STATE_OACK;
>  		TftpServerPort = src;
>  		/*
>   
Applied to net repo.

thanks,
Ben