cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] TFTP portability fix

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 8 May 2006 23:06:21 +0200 (CEST)

On Mon, 8 May 2006, Dan Fandrich wrote:

> * There are references to errno instead of Curl_sockerrno(), causing incorrect
> error messages on Windows

> * Failures in EBCDIC conversion are not propagated up to the caller but are
> silently ignored
>
> I've checked in fixes for the first four issues.

Great! I think I've just fixed theses two above leaving us with this one
remaining:

> * The file name includes a leading / which violates RFC3617. Doing something
> similar to ftp, where two slashes after the host name means an absolute
> reference seems a reasonable extension to fix this.

I agree. And the simple fix to make it do this is to simply skip the first
character of the filename when including it in the TFTP package, isn't it?

-- 
  Commercial curl and libcurl Technical Support: http://haxx.se/curl.html
Received on 2006-05-08