netdev - Re: [PATCH ethtool 2/3] Add IPv6 support to NFC

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: <1457887406.3331.31.camel@decadent.org.uk>
Date:	Sun, 13 Mar 2016 16:43:26 +0000
From:	Ben Hutchings <ben@...adent.org.uk>
To:	Edward Cree <ecree@...arflare.com>, Ben Hutchings <bwh@...nel.org>
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH ethtool 2/3] Add IPv6 support to NFC

On Mon, 2016-02-15 at 14:59 +0000, Edward Cree wrote:
> Signed-off-by: Edward Cree <ecree@...arflare.com>
[...]
> @@ -950,6 +1154,19 @@ static int rxclass_get_mask(char *str, unsigned char *p,
>                 *(__be32 *)&p[opt->moffset] = ~val;
>                 break;
>         }
> +       case OPT_IP6: {
> +               __be32 val[4];
> +               int i;
> +               err = rxclass_get_ipv6(str, val);
> +               if (err)
> +                       return -1;
> +               for (i = 0; i < 4; i++) {
> +                       ((__be32 *)&p[opt->offset])[i] = val[i];
> +                       if (opt->moffset >= 0)
> +                               ((__be32 *)&p[opt->moffset])[i] = ~val[i];

This pointer arithmetic looks terrible.  I think memcpy() would be much
clearer here.

[...]
> The information contained in this message is confidential and is
> intended for the addressee(s) only. If you have received this message
> in error, please notify the sender immediately and delete the
> message. Unless you are an addressee (or authorized to receive for an
> addressee), you may not use, copy or disclose to anyone this message
> or any information contained in this message. The unauthorized use,
> disclosure, copying or alteration of this message is strictly
> prohibited.

I won't apply patches labelled as "confidential".  You need to stop
including this nonsense in your public messages (I thought you fixed
this once before).

Ben.

-- 
Ben Hutchings
If at first you don't succeed, you're doing about average.
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)