netdev - Re: [BUG] "$ ss -a" incorrectly displays raw sockets as udp sockets

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 09 Jul 2015 16:50:06 +0200
From:	Nikolay Aleksandrov <nikolay@...ulusnetworks.com>
To:	Miha Marolt <miham@...ondsemi.com>, netdev@...r.kernel.org,
	vadim4j@...il.com
Subject: Re: [BUG] "$ ss -a" incorrectly displays raw sockets as udp sockets

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

View attachment "ss-raw.patch" of type "text/x-patch" (595 bytes)