From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 43AED59CB1 for ; Tue, 12 Apr 2016 13:15:57 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 40E3DE086C; Tue, 12 Apr 2016 13:15:55 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id BC034E086C for ; Tue, 12 Apr 2016 13:15:54 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 709AB340B57 for ; Tue, 12 Apr 2016 13:15:53 +0000 (UTC) Received: from localhost.localdomain (localhost [127.0.0.1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id CE282201 for ; Tue, 12 Apr 2016 13:15:50 +0000 (UTC) From: "Sergey Popov" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Sergey Popov" Message-ID: <1460466910.ca6d97524c94cc87abce4d9b3281d6e6be646838.pinkbyte@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: net-misc/quagga/files/, net-misc/quagga/ X-VCS-Repository: repo/gentoo X-VCS-Files: net-misc/quagga/files/quagga-1.0.20160315-ripd-null-pointer-fix.patch net-misc/quagga/quagga-1.0.20160315-r1.ebuild net-misc/quagga/quagga-1.0.20160315.ebuild X-VCS-Directories: net-misc/quagga/ net-misc/quagga/files/ X-VCS-Committer: pinkbyte X-VCS-Committer-Name: Sergey Popov X-VCS-Revision: ca6d97524c94cc87abce4d9b3281d6e6be646838 X-VCS-Branch: master Date: Tue, 12 Apr 2016 13:15:50 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Archives-Salt: 9eacc14c-5d73-4b8e-b987-b857ee3f93c8 X-Archives-Hash: 9bc9529f5aae5118b55e2dde32c3a989 commit: ca6d97524c94cc87abce4d9b3281d6e6be646838 Author: Sergey Popov gentoo org> AuthorDate: Tue Apr 12 13:02:00 2016 +0000 Commit: Sergey Popov gentoo org> CommitDate: Tue Apr 12 13:15:10 2016 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ca6d9752 net-misc/quagga: revision bump Fix NULL pointer dereference in ripd Commited straight to stable Reported-by: Sebastian Kricner tuxwave.net> Acked-by: Sergey Popov gentoo.org> Gentoo-Bug: 579386 Package-Manager: portage-2.2.28 RepoMan-Options: --force ...quagga-1.0.20160315-ripd-null-pointer-fix.patch | 40 ++++++++++++++++++++++ ...160315.ebuild => quagga-1.0.20160315-r1.ebuild} | 5 ++- 2 files changed, 44 insertions(+), 1 deletion(-) diff --git a/net-misc/quagga/files/quagga-1.0.20160315-ripd-null-pointer-fix.patch b/net-misc/quagga/files/quagga-1.0.20160315-ripd-null-pointer-fix.patch new file mode 100644 index 0000000..e48c304 --- /dev/null +++ b/net-misc/quagga/files/quagga-1.0.20160315-ripd-null-pointer-fix.patch @@ -0,0 +1,40 @@ +The rip_output_process function dereferenced a NULL +pointer. Core file examination showed that tmp_rinfo +was NULL on line 2435. Looking at the last diff +associated with this commit, it was obvious that +a formating mistake had been made in the loop over +the route nodes list of possible paths. + +Signed-off-by: Donald Sharp +Reported-by: Sebastian Kricner +--- + ripd/ripd.c | 14 ++++++++------ + 1 file changed, 8 insertions(+), 6 deletions(-) + +diff --git a/ripd/ripd.c b/ripd/ripd.c +index 82b1ada..0beb0e6 100644 +--- a/ripd/ripd.c ++++ b/ripd/ripd.c +@@ -2429,12 +2429,14 @@ rip_output_process (struct connected *ifc, struct sockaddr_in *to, + struct rip_info *tmp_rinfo = NULL; + + for (ALL_LIST_ELEMENTS_RO (list, listnode, tmp_rinfo)) +- if (tmp_rinfo->type == ZEBRA_ROUTE_RIP && +- tmp_rinfo->ifindex == ifc->ifp->ifindex) +- rinfo->metric_out = RIP_METRIC_INFINITY; +- if (tmp_rinfo->type == ZEBRA_ROUTE_CONNECT && +- prefix_match((struct prefix *)p, ifc->address)) +- rinfo->metric_out = RIP_METRIC_INFINITY; ++ { ++ if (tmp_rinfo->type == ZEBRA_ROUTE_RIP && ++ tmp_rinfo->ifindex == ifc->ifp->ifindex) ++ rinfo->metric_out = RIP_METRIC_INFINITY; ++ if (tmp_rinfo->type == ZEBRA_ROUTE_CONNECT && ++ prefix_match((struct prefix *)p, ifc->address)) ++ rinfo->metric_out = RIP_METRIC_INFINITY; ++ } + } + + /* Prepare preamble, auth headers, if needs be */ +-- +1.9.1 diff --git a/net-misc/quagga/quagga-1.0.20160315.ebuild b/net-misc/quagga/quagga-1.0.20160315-r1.ebuild similarity index 96% rename from net-misc/quagga/quagga-1.0.20160315.ebuild rename to net-misc/quagga/quagga-1.0.20160315-r1.ebuild index 949953c..b328a70 100644 --- a/net-misc/quagga/quagga-1.0.20160315.ebuild +++ b/net-misc/quagga/quagga-1.0.20160315-r1.ebuild @@ -34,7 +34,10 @@ DEPEND="${COMMON_DEPEND} RDEPEND="${COMMON_DEPEND} sys-apps/iproute2" -PATCHES=( "${FILESDIR}/${PN}-0.99.22.4-ipctl-forwarding.patch" ) +PATCHES=( + "${FILESDIR}/${PN}-0.99.22.4-ipctl-forwarding.patch" + "${FILESDIR}/${PN}-1.0.20160315-ripd-null-pointer-fix.patch" +) DISABLE_AUTOFORMATTING=1 DOC_CONTENTS="Sample configuration files can be found in /usr/share/doc/${PF}/samples