Summary: | net-misc/quagga-1.2.1: test failures on sparc | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Rolf Eike Beer <eike> |
Component: | Current packages | Assignee: | Sergey Popov (RETIRED) <pinkbyte> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | TESTFAILURE |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://gogs.quagga.net/Quagga/quagga/commit/adda534f95ec87206c9dfd1b3bae05221dc29730 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build.log |
Description
Rolf Eike Beer
![]() Created attachment 506746 [details]
build.log
What does that mean? Ignore those errors and stabilize it anyway? (In reply to Rolf Eike Beer from comment #2) > What does that mean? Ignore those errors and stabilize it anyway? Testing errors can not prevent security stabilization per our policy It's actually not a testcase error, but a real code error. This is the fix: diff --git a/bgpd/bgp_open.c b/bgpd/bgp_open.c index 28004230..d9ec4bef 100644 --- a/bgpd/bgp_open.c +++ b/bgpd/bgp_open.c @@ -120,7 +120,8 @@ bgp_capability_vty_out (struct vty *vty, struct peer *peer) static void bgp_capability_mp_data (struct stream *s, struct capability_mp_data *mpc) { - mpc->afi = stream_getw (s); + afi_t afi = stream_getw (s); + memcpy(&mpc->afi, &afi, sizeof(mpc->afi)); mpc->reserved = stream_getc (s); mpc->safi = stream_getc (s); } Fix is upstream, should be in next (point) release. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ab71ee7a03166f69b15849ab336ed5056aa0b8d0 commit ab71ee7a03166f69b15849ab336ed5056aa0b8d0 Author: Sergey Popov <pinkbyte@gentoo.org> AuthorDate: 2018-01-25 16:03:14 +0000 Commit: Sergey Popov <pinkbyte@gentoo.org> CommitDate: 2018-01-25 16:03:28 +0000 net-misc/quagga: revision bump Backport fixes for test failures on sparc and incorrect ipv6 route handling Closes: https://bugs.gentoo.org/638906 Closes: https://bugs.gentoo.org/641736 Package-Manager: Portage-2.3.19, Repoman-2.3.6 net-misc/quagga/files/quagga-1.2.2-ipv6-bgp.patch | 42 +++++++ .../quagga/files/quagga-1.2.2-sparc-tests.patch | 31 +++++ net-misc/quagga/quagga-1.2.2-r1.ebuild | 137 +++++++++++++++++++++ 3 files changed, 210 insertions(+) Just for the record: this was no test bug, but a real library bug. (In reply to Rolf Eike Beer from comment #7) > Just for the record: this was no test bug, but a real library bug. We know that(actually it was pretty clear from your patch draft - it did not fix testsuite, but the actual code), but we prefer to stick with upstream decision - commit adda534f95ec87206c9dfd1b3bae05221dc29730 says only about test failures. |