RE: [PATCH 3.8-stable] netfilter: nfnetlink_queue: fix error return code in nfnetlink_queue_init()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Pablo Neira Ayuso
> > From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
> >
> > This patch looks like it should be in the 3.8-stable tree, should we
> apply
> > it?
> 
> Please, hold on with this. I'm preparing a -stable batch.


I see. Thanks.

Best Regards.

> -----Original Message-----
> From: stable-owner@xxxxxxxxxxxxxxx [mailto:stable-owner@xxxxxxxxxxxxxxx]
> On Behalf Of Pablo Neira Ayuso
> Sent: Saturday, April 13, 2013 7:08 PM
> To: Jonghwan Choi
> Cc: Wei Yongjun; stable@xxxxxxxxxxxxxxx; netfilter@xxxxxxxxxxxxxxx;
> Jonghwan Choi
> Subject: Re: [PATCH 3.8-stable] netfilter: nfnetlink_queue: fix error
> return code in nfnetlink_queue_init()
> 
> On Sat, Apr 13, 2013 at 12:14:23AM +0900, Jonghwan Choi wrote:
> > From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
> >
> > This patch looks like it should be in the 3.8-stable tree, should we
> apply
> > it?
> 
> Please, hold on with this. I'm preparing a -stable batch.
> 
> > ------------------
> >
> > From: "Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>"
> >
> > commit 558724a5b2a73ad0c7638e21e8dffc419d267b6c upstream
> >
> > Fix to return a negative error code from the error handling
> > case instead of 0, as returned elsewhere in this function.
> >
> > Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
> > Signed-off-by: Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx>
> > Signed-off-by: Jonghwan Choi <jhbird.choi@xxxxxxxxxxx>
> > ---
> >  net/netfilter/nfnetlink_queue_core.c |    4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/net/netfilter/nfnetlink_queue_core.c
> b/net/netfilter/nfnetlink_queue_core.c
> > index 3158d87..39006c9 100644
> > --- a/net/netfilter/nfnetlink_queue_core.c
> > +++ b/net/netfilter/nfnetlink_queue_core.c
> > @@ -1064,8 +1064,10 @@ static int __init nfnetlink_queue_init(void)
> >
> >  #ifdef CONFIG_PROC_FS
> >  	if (!proc_create("nfnetlink_queue", 0440,
> > -			 proc_net_netfilter, &nfqnl_file_ops))
> > +			 proc_net_netfilter, &nfqnl_file_ops)) {
> > +		status = -ENOMEM;
> >  		goto cleanup_subsys;
> > +	}
> >  #endif
> >
> >  	register_netdevice_notifier(&nfqnl_dev_notifier);
> > --
> > 1.7.10.4
> >
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]