Re: [Lynx-dev] ..on SOCKS5 support

[Thomas Dickey]

On Mon, Sep 16, 2019 at 01:16:48AM +0000, Thorsten Glaser wrote:
> Thomas Dickey dixit:
> 
> >well... I'll have to set up something for testing
> 
> ssh -D starts a SOCKS5 server on the SSH client side AIUI.

thanks (will see)
 
> >but here's something more like the style of lynx
> 
> Please…
> 
> -       /* C99 */  {
> -           unsigned short x;   /* XXX 16-bit? */
> -
> -           x = htons(socks5_port);
> -           memcpy(&pbuf[i], (unsigned char *) &x, sizeof x);
> -           i += (unsigned) sizeof(x);
> -       }
> +       pbuf[i++] = (((unsigned)socks5_port) >> 8) & 0xFF;
> +       pbuf[i++] = ((unsigned)socks5_port) & 0xFF;
> 
> (or just make socks5_port unsigned int, if you can)

I missed that (most of my attention was on the compiler warnings).
 
> This should even be optimisable by compilers.

-- 
Thomas E. Dickey <address@hidden>
https://invisible-island.net
ftp://ftp.invisible-island.net

signature.asc
Description: PGP signature