netdev - Re: [PATCH] iputils ping: add (non-raw) ICMP socket support

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]

Message-ID: <1749676.NUxJ1eyH1x@msg-id>
Date:	Wed, 08 Apr 2015 11:08:45 +0200
From:	Salvatore Mesoraca <s.mesoraca16@...il.com>
To:	YOSHIFUJI Hideaki <hideaki.yoshifuji@...aclelinux.com>,
	netdev@...r.kernel.org
Cc:	Vasiliy Kulikov <segooon@...il.com>,
	Tyler Hicks <tyhicks@...onical.com>
Subject: Re: [PATCH] iputils ping: add (non-raw) ICMP socket support

YOSHIFUJI Hideaki wrote:
> Hi,
Hi,
thank you for thaking to the time to answer

> Salvatore Mesoraca wrote:
> > From: Vasiliy Kulikov <segooon@...il.com>
> > 
> > This patch adds non-raw IPPROTO_ICMP socket kind support that was added
> > to the Linux 3.0.  The patch is backward-compatible: if ICMP socket kind
> > is not enabled in the kernel (either in case of an old kernel or
> > explicitly disabled via /proc/sys/net/ipv4/ping_group_range), ping uses
> > old privileged raw sockets as a fallback.
> > 
> > This patch is going to be included in Ubuntu 15.10 and it is already
> > included in Gentoo stable tree (at the moment of the writing ping has
> > CAP_NET_RAW still enabled by default) it is also included in OpenWall
> > since 2011.
> > This patch also tries to sneak in a fix for a missing colon in a printf.
> > I've tested it on Linux 3.17.7 and it worked without issues.
> 
> Please do not mix changes in a single commit.
> Thanks.

I had some doubt about that additional change (it is so small that I don't 
think that anyone will ever make a separate patch for it), but it was present 
in the original patch and I decided to add it anyway and wait for feedback.
I'll delete it.
Thank you again for you comment.


--
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