Re: [PATCH] crypto: user - Allow CRYPTO_MSG_GETALG without CAP_NET_ADMIN

[Marek Vasut <marex@denx.de>]

On Wednesday, April 30, 2014 at 09:23:40 PM, Matthias-Christian Ott wrote:
> On 04/28/14 23:37, Marek Vasut wrote:
> > On Friday, April 25, 2014 at 12:51:06 AM, Matthias-Christian Ott wrote:
> >> CRYPTO_USER requires CAP_NET_ADMIN for all operations. Most information
> >> provided by CRYPTO_MSG_GETALG is also accessible through /proc/modules
> >> and AF_ALG. CRYPTO_MSG_GETALG should not require CAP_NET_ADMIN so that
> >> processes without CAP_NET_ADMIN can use CRYPTO_MSG_GETALG to get cipher
> >> details, such as cipher priorities, for AF_ALG.
> >> 
> >> Signed-off-by: Matthias-Christian Ott <ott@mirix.org>
> >> ---
> >> 
> >>  crypto/crypto_user.c | 12 +++++++++---
> >>  1 file changed, 9 insertions(+), 3 deletions(-)
> > 
> > Can you please submit the patch using git send-email so we can properly
> > review it?
> 
> git-send-email or more specifically Net::SMTP only works with IPv4. The
> SMTP server I use for submission only listens IPv6 addresses. Moreover,
> TLS seems broken. I patched Net::SMTP to use IO::Socket::INET6 but gave
> up because of the TLS issues and used git format-patch and git imap-send.

You can always set up a separate mailserver or use one of the many free-to-use 
mailservers to follow the agreed-upon submission process, right?

Of course, patches for Net::SMTP are welcome.

Best regards,
Marek Vasut