linux-kernel - Re: [PATCH] sctp: Remove some redundant code

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: <28e1309f-a084-2090-ce2a-a43c013ddc8e@wanadoo.fr>
Date:   Thu, 15 Sep 2016 21:32:18 +0200
From:   Christophe JAILLET <christophe.jaillet@...adoo.fr>
To:     Marcelo <marcelo.leitner@...il.com>
Cc:     vyasevich@...il.com, nhorman@...driver.com, davem@...emloft.net,
        linux-sctp@...r.kernel.org, linux-kernel@...r.kernel.org,
        kernel-janitors@...r.kernel.org
Subject: Re: [PATCH] sctp: Remove some redundant code

Le 15/09/2016 à 19:34, Marcelo a écrit :
> On Sun, Sep 04, 2016 at 11:38:13AM +0200, Christophe JAILLET wrote:
>> In commit 311b21774f13 ("sctp: simplify sk_receive_queue locking"), a call
>> to 'skb_queue_splice_tail_init()' has been made explicit. Previously it was
>> hidden in 'sctp_skb_list_tail()'
>>
>> Now, the code around it looks redundant. The '_init()' part of
>> 'skb_queue_splice_tail_init()' should alreday do the same.
>>
>> Signed-off-by: Christophe JAILLET <christophe.jaillet@...adoo.fr>
> I just noticed you didn't include netdev@ in the destinations.
>
> Is your patch going to be tracked somewhere else? It's not showing up in
> http://patchwork.ozlabs.org/project/netdev/list/?submitter=66309&state=*&q=&archive=&delegate=
>
My fault, get_maintainer lists netdev@ but I missed it.

Should I resubmit or is it ok?

CJ