> -----Original Message----- > From: Gao Feng [mailto:gfree.wind@xxxxxxxxxxx] > Sent: Friday, April 7, 2017 6:16 AM > To: 'Pablo Neira Ayuso' <pablo@xxxxxxxxxxxxx> > Cc: 'netfilter-devel@xxxxxxxxxxxxxxx' <netfilter-devel@xxxxxxxxxxxxxxx>; 'Gao > Feng' <fgao@xxxxxxxxxx> > Subject: RE: [PATCH nf-next v2 1/1] netfilter: nat: Correct the return value > check for nat mangled packet > > Hi Palbo, > > > -----Original Message----- > > From: Pablo Neira Ayuso [mailto:pablo@xxxxxxxxxxxxx] > > Sent: Friday, April 7, 2017 3:25 AM > > To: gfree.wind@xxxxxxxxxxx > > Cc: netfilter-devel@xxxxxxxxxxxxxxx; Gao Feng <fgao@xxxxxxxxxx> > > Subject: Re: [PATCH nf-next v2 1/1] netfilter: nat: Correct the return > > value check for nat mangled packet > > > > On Thu, Apr 06, 2017 at 09:18:19PM +0200, Pablo Neira Ayuso wrote: > > > On Mon, Mar 27, 2017 at 11:12:08PM +0800, gfree.wind@xxxxxxxxxxx > > wrote: > > > > From: Gao Feng <fgao@xxxxxxxxxx> > > > > > > > > The return value type of function nf_nat_mangle_tcp/udp_packet is > > > > int, but actually it is used as bool type. And most codes follow > > > > this rule, for example, the sip, h323, and ftp. But some codes > > > > treat the return value as NF_ACCEPT and NF_DROP, like amanda and irc. > > > > > > > > Now use the bool type instead of the int to make it clear. > > > > > > One spot still not converted. > > > > > > I'm going to collapse this patch to yours. Please review so you > > > don't have to send a v3. > > > > And if this chunk looks good to you, I'll apply this patch with this > > title: > > > > netfilter: nf_nat: nf_nat_mangle_{udp,tcp}_packet returns Boolean > > Thanks, it is ok to me. > > Best Regards > Feng Sorry, I misspelled your name in the last email. Best Regards Feng -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
