Message ID | 1396139975-5314-1-git-send-email-sasha.levin@oracle.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: Sasha Levin <sasha.levin@oracle.com> Date: Sat, 29 Mar 2014 20:39:35 -0400 > Binding might result in a NULL device which is later dereferenced > without checking. > > Signed-off-by: Sasha Levin <sasha.levin@oracle.com> Applied, thanks Sasha. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi David, On Mon, Mar 31, 2014 at 04:26:07PM -0400, David Miller wrote: > From: Sasha Levin <sasha.levin@oracle.com> > Date: Sat, 29 Mar 2014 20:39:35 -0400 > > > Binding might result in a NULL device which is later dereferenced > > without checking. > > > > Signed-off-by: Sasha Levin <sasha.levin@oracle.com> > > Applied, thanks Sasha. > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html This is very similar to c2349758acf1874e4c2b93fe41d072336f1a31d0 ("rds: prevent dereference of a NULL device"), which you've queued for stable. Could you please queue this one for stable as well? (Looks like it even has CVE-2014-2678 assigned to it.) Cheers, -- Luís -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Luís Henriques <luis.henriques@canonical.com> Date: Fri, 4 Apr 2014 15:31:36 +0100 > Could you please queue this one for stable as well? (Looks like it even > has CVE-2014-2678 assigned to it.) Done. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Apr 04, 2014 at 10:44:26AM -0400, David Miller wrote: > From: Luís Henriques <luis.henriques@canonical.com> > Date: Fri, 4 Apr 2014 15:31:36 +0100 > > > Could you please queue this one for stable as well? (Looks like it even > > has CVE-2014-2678 assigned to it.) > > Done. That was fast! Thanks :) Cheers, -- Luís -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/net/rds/iw.c b/net/rds/iw.c index 7826d46..0298920 100644 --- a/net/rds/iw.c +++ b/net/rds/iw.c @@ -239,7 +239,8 @@ static int rds_iw_laddr_check(__be32 addr) ret = rdma_bind_addr(cm_id, (struct sockaddr *)&sin); /* due to this, we will claim to support IB devices unless we check node_type. */ - if (ret || cm_id->device->node_type != RDMA_NODE_RNIC) + if (ret || !cm_id->device || + cm_id->device->node_type != RDMA_NODE_RNIC) ret = -EADDRNOTAVAIL; rdsdebug("addr %pI4 ret %d node type %d\n",
Binding might result in a NULL device which is later dereferenced without checking. Signed-off-by: Sasha Levin <sasha.levin@oracle.com> --- net/rds/iw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)