[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/1] slirp: add SOCKS5 support
From: |
Samuel Thibault |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/1] slirp: add SOCKS5 support |
Date: |
Sat, 6 May 2017 01:27:38 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
Hello,
Laurent Vivier, on sam. 06 mai 2017 00:48:33 +0200, wrote:
> @@ -617,6 +622,10 @@ void slirp_pollfds_poll(GArray *pollfds, int
> select_error)
> * Check sockets for reading
> */
> else if (revents & (G_IO_IN | G_IO_HUP | G_IO_ERR)) {
> + if (so->so_state & SS_ISFCONNECTING) {
> + socks5_recv(so->s, &so->so_proxy_state);
> + continue;
> + }
Again, I don't see how this can work with both socks5 case and
non-socks5 case. Don't we need to somehow check for the type of socket
before calling socks5_recv?
> @@ -645,11 +654,19 @@ void slirp_pollfds_poll(GArray *pollfds, int
> select_error)
> /*
> * Check for non-blocking, still-connecting sockets
> */
> - if (so->so_state & SS_ISFCONNECTING) {
> - /* Connected */
> - so->so_state &= ~SS_ISFCONNECTING;
>
> - ret = send(so->s, (const void *) &ret, 0, 0);
> + if (so->so_state & SS_ISFCONNECTING) {
> + ret = socks5_send(so->s, slirp->proxy_user,
Ditto.
> diff --git a/slirp/socks5.c b/slirp/socks5.c
> new file mode 100644
> index 0000000..2bba045
> --- /dev/null
> +++ b/slirp/socks5.c
> @@ -0,0 +1,371 @@
In v2 of the patch, this was said to have "some parts from nmap/ncat
GPLv2". Is that really not true any more? If any part of the file is
not original, it *has* to wear proper copyright notices, otherwise it's
copyright infrigement.
Also, see the bot build error report: <sys/socket.h> doesn't exist on
windows,
#include <windows.h>
#include <winsock2.h>
#include <ws2tcpip.h>
should be used instead.
Samuel
Re: [Qemu-devel] [PATCH v4 0/1] slirp: add SOCKS5 support, no-reply, 2017/05/05
Re: [Qemu-devel] [PATCH v4 0/1] slirp: add SOCKS5 support, no-reply, 2017/05/05