lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Thu, 9 Jul 2015 18:15:06 +0300
From:	Vadim Kochan <vadim4j@...il.com>
To:	Miha Marolt <miham@...ondsemi.com>
Cc:	Nikolay Aleksandrov <nikolay@...ulusnetworks.com>,
	Vadim Kochan <vadim4j@...il.com>, netdev@...r.kernel.org
Subject: Re: [BUG] "$ ss -a" incorrectly displays raw sockets as udp sockets

On Thu, Jul 09, 2015 at 05:09:27PM +0200, Miha Marolt wrote:
> 
> 
> On 07/09/2015 04:57 PM, Nikolay Aleksandrov wrote:
> >On 07/09/2015 04:55 PM, Vadim Kochan wrote:
> >>On Thu, Jul 09, 2015 at 04:50:06PM +0200, Nikolay Aleksandrov wrote:
> >>>On 07/09/2015 04:13 PM, Miha Marolt wrote:
> >>>>Hi!
> >>>>
> >>>>I hope this is the right place to reports bugs. I apologize if it isn't.
> >>>>
> >>>>I have written a C program (see below for source code) that opens a raw socket on CentOS 7.1 Linux and binds it to some address (it doesn't use the port that I supplied, but that is not the point here). The "netstat" program correctly recognizes the socket as "raw", while "ss" program says it is "udp". Here are the relevant lines from the ss and netstat commands:
> >>>>
> >>>>$ netstat -an
> >>>>raw        0      0 127.0.0.1:6             0.0.0.0:* 7
> >>>>
> >>>>$ ./ss -an
> >>>>udp    UNCONN     21569  0      127.0.0.1:6 *:*
> >>>>
> >>>>Here is the version information
> >>>>
> >>>>$ netstat --version  # From CentOS 7.1.
> >>>>net-tools 2.10-alpha
> >>>>
> >>>>$ ./ss --version  # Built from git.
> >>>>ss utility, iproute2-ss150626
> >>>>
> >>>>
> >>>>C source follows. If you store it in "main.c", then compile it with "$ gcc main.c -o main" and then run it by executing "$ sudo ./main".
> >>>>
> >>>>#include <arpa/inet.h>
> >>>>#include <assert.h>
> >>>>#include <sys/socket.h>
> >>>>#include <stdio.h>
> >>>>#include <unistd.h>
> >>>>
> >>>>
> >>>>int main(void)
> >>>>{
> >>>>     // Create a raw socket.
> >>>>     int sock = socket(AF_INET, SOCK_RAW, IPPROTO_TCP);
> >>>>     if (sock == -1) { perror(NULL); goto exc_cleanup; }
> >>>>
> >>>>     // Bind socket to an address.
> >>>>     struct sockaddr_in addr;
> >>>>     addr.sin_family = AF_INET;
> >>>>     inet_pton(AF_INET, "127.0.0.1", &addr.sin_addr);
> >>>>     addr.sin_port = htons(27183);
> >>>>
> >>>>     int rc = bind(sock, (struct sockaddr*)&addr, sizeof(addr));
> >>>>     if (rc != 0) { perror(NULL); goto exc_cleanup; }
> >>>>
> >>>>     // Wait until user presses <ENTER>.
> >>>>     printf("\nPress <ENTER> to quit the program.\n");
> >>>>     getchar();
> >>>>
> >>>>exc_cleanup:
> >>>>     assert(!close(sock));
> >>>>}
> >>>>
> >>>>
> >>>>Best regards,
> >>>>Miha
> >>>
> >>>Hi,
> >>>I think this was changed by commit:
> >>>8250bc9ff4e5 ("ss: Unify inet sockets output")
> >>>Because dgram_show_line() is used for both UDP and RAW sockets
> >>>IPPROTO_UDP is used for both now and proto_name() returns "udp".
> >>>CCed the patch author and attached a possible solution.
> >>>
> >>>Cheers,
> >>>  Nik
> >>>diff --git a/misc/ss.c b/misc/ss.c
> >>>index 870cad185341..4de77e92c319 100644
> >>>--- a/misc/ss.c
> >>>+++ b/misc/ss.c
> >>>@@ -1554,6 +1554,8 @@ out:
> >>>  static char *proto_name(int protocol)
> >>>  {
> >>>  	switch (protocol) {
> >>>+	case IPPROTO_RAW:
> >>>+		return "raw";
> >>>  	case IPPROTO_UDP:
> >>>  		return "udp";
> >>>  	case IPPROTO_TCP:
> >>>@@ -2398,7 +2400,7 @@ static int dgram_show_line(char *line, const struct filter *f, int family)
> >>>  	if (n < 9)
> >>>  		opt[0] = 0;
> >>>-	inet_stats_print(&s, IPPROTO_UDP);
> >>>+	inet_stats_print(&s, dg_proto == UDP_PROTO ? IPPROTO_UDP : IPPROTO_RAW);
> >>>  	if (show_details && opt[0])
> >>>  		printf(" opt:\"%s\"", opt);
> >>Yeah, it fixed the issue, just tested the fix.
> >>
> >Great, I'll submit it in a minute.
> >Thanks for testing!
> 
> Thank you very much for the fast fix!
> 
> I've found another bug: command "$ ss" doesn't display only tcp sockets as
> the man page ss(8) says (file man/man8/ss.8 in iproute2 git repo). It also
> displays e.g. unix sockets. The man page says "When no option is used ss
> displays a list of open non-listening TCP sockets that have established
> connection.".
> 
> 
> Regards,
> Miha

That seems OK, I see that unix stream sockets are displayed which are
connection oriented, and UDP can also have established state.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists