From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied by user verbs cmds Date: Fri, 14 Jul 2017 08:49:08 +0200 Message-ID: <20170714064908.GC16816@kroah.com> References: <20170713153957.515045341@linuxfoundation.org> <20170713153959.566717957@linuxfoundation.org> <5C5647B5F4794941BAAC07CF28785CBC33E4E81F@fmsmsx101.amr.corp.intel.com> <20170713162540.GB22870@kroah.com> <5C5647B5F4794941BAAC07CF28785CBC33E4EC25@fmsmsx101.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <5C5647B5F4794941BAAC07CF28785CBC33E4EC25-96pTJSsuoYSkrb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Ismail, Mustafa" Cc: "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Yevgeny Kliteynik , Tziporet Koren , Alex Polak , Boris Pismenny , Leon Romanovsky , Doug Ledford List-Id: linux-rdma@vger.kernel.org On Thu, Jul 13, 2017 at 06:44:38PM +0000, Ismail, Mustafa wrote: > > -----Original Message----- > > From: Greg Kroah-Hartman [mailto:gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org] > > Sent: Thursday, July 13, 2017 11:26 AM > > To: Ismail, Mustafa > > Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; > > stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Yevgeny Kliteynik ; > > Tziporet Koren ; Alex Polak > > ; Boris Pismenny ; Leon > > Romanovsky ; Doug Ledford > > Subject: Re: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied > > by user verbs cmds > > > > On Thu, Jul 13, 2017 at 03:54:28PM +0000, Ismail, Mustafa wrote: > > > > Subject: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied > > > > by user verbs cmds > > > > > > > > 4.4-stable review patch. If anyone has any objections, please let me > > know. > > > > > > Yes, this breaks modify qp. > > > See https://patchwork.kernel.org/patch/9830663/ > > > > I don't understand this response at all, sorry. > > > > What should I do about this? Is this patch alone a problem? Is there some > > other patch I should apply that is in Linus's tree? Where is the problem, > > only in this old release? > > > Applying this patch will break RDMA functionality with respect to modify_qp. > Specifically this part: > + if (cmd.port_num < rdma_start_port(ib_dev) || > + cmd.port_num > rdma_end_port(ib_dev)) > + return -EINVAL; > + > The cmd.port_num is only valid if (cmd->base.attr_mask & IB_QP_PORT). > So the above can be fixed with: > if ((cmd->base.attr_mask & IB_QP_PORT) && > (cmd.port_num < rdma_start_port(ib_dev) || > cmd.port_num > rdma_end_port(ib_dev))) > return -EINVAL; > > A version of this patch is in Linus's tree and it has the same problem: > commit 5ecce4c9b17bed4dc9cb58bfb10447307569b77b "RDMA/uverbs: Check port number supplied by user verbs cmds" > > We will be submitting a patch to fix this shortly. Ok, so we will be "bug compatible" with Linus's tree then, right? :) Should I hold off on applying this now, or just stay in sync and wait for your fixes to land in Linus's tree? I vote stay-in-sync, as the other stable trees are also "broken" in the same way right now... thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753201AbdGNGtP (ORCPT ); Fri, 14 Jul 2017 02:49:15 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:36806 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751166AbdGNGtN (ORCPT ); Fri, 14 Jul 2017 02:49:13 -0400 Date: Fri, 14 Jul 2017 08:49:08 +0200 From: Greg Kroah-Hartman To: "Ismail, Mustafa" Cc: "linux-kernel@vger.kernel.org" , "linux-rdma@vger.kernel.org" , "stable@vger.kernel.org" , Yevgeny Kliteynik , Tziporet Koren , Alex Polak , Boris Pismenny , Leon Romanovsky , Doug Ledford Subject: Re: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied by user verbs cmds Message-ID: <20170714064908.GC16816@kroah.com> References: <20170713153957.515045341@linuxfoundation.org> <20170713153959.566717957@linuxfoundation.org> <5C5647B5F4794941BAAC07CF28785CBC33E4E81F@fmsmsx101.amr.corp.intel.com> <20170713162540.GB22870@kroah.com> <5C5647B5F4794941BAAC07CF28785CBC33E4EC25@fmsmsx101.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5C5647B5F4794941BAAC07CF28785CBC33E4EC25@fmsmsx101.amr.corp.intel.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 13, 2017 at 06:44:38PM +0000, Ismail, Mustafa wrote: > > -----Original Message----- > > From: Greg Kroah-Hartman [mailto:gregkh@linuxfoundation.org] > > Sent: Thursday, July 13, 2017 11:26 AM > > To: Ismail, Mustafa > > Cc: linux-kernel@vger.kernel.org; linux-rdma@vger.kernel.org; > > stable@vger.kernel.org; Yevgeny Kliteynik ; > > Tziporet Koren ; Alex Polak > > ; Boris Pismenny ; Leon > > Romanovsky ; Doug Ledford > > Subject: Re: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied > > by user verbs cmds > > > > On Thu, Jul 13, 2017 at 03:54:28PM +0000, Ismail, Mustafa wrote: > > > > Subject: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied > > > > by user verbs cmds > > > > > > > > 4.4-stable review patch. If anyone has any objections, please let me > > know. > > > > > > Yes, this breaks modify qp. > > > See https://patchwork.kernel.org/patch/9830663/ > > > > I don't understand this response at all, sorry. > > > > What should I do about this? Is this patch alone a problem? Is there some > > other patch I should apply that is in Linus's tree? Where is the problem, > > only in this old release? > > > Applying this patch will break RDMA functionality with respect to modify_qp. > Specifically this part: > + if (cmd.port_num < rdma_start_port(ib_dev) || > + cmd.port_num > rdma_end_port(ib_dev)) > + return -EINVAL; > + > The cmd.port_num is only valid if (cmd->base.attr_mask & IB_QP_PORT). > So the above can be fixed with: > if ((cmd->base.attr_mask & IB_QP_PORT) && > (cmd.port_num < rdma_start_port(ib_dev) || > cmd.port_num > rdma_end_port(ib_dev))) > return -EINVAL; > > A version of this patch is in Linus's tree and it has the same problem: > commit 5ecce4c9b17bed4dc9cb58bfb10447307569b77b "RDMA/uverbs: Check port number supplied by user verbs cmds" > > We will be submitting a patch to fix this shortly. Ok, so we will be "bug compatible" with Linus's tree then, right? :) Should I hold off on applying this now, or just stay in sync and wait for your fixes to land in Linus's tree? I vote stay-in-sync, as the other stable trees are also "broken" in the same way right now... thanks, greg k-h