Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 761885 - net-p2p/retroshare-0.6.5-r2: stabilization and cleanup (was: depends on vulnerable net-libs/libupnp)
Summary: net-p2p/retroshare-0.6.5-r2: stabilization and cleanup (was: depends on vulne...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Stabilization (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gioacchino Mazzurco
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: CVE-2020-13848
  Show dependency tree
 
Reported: 2020-12-27 08:06 UTC by John Helmert III
Modified: 2021-03-12 16:01 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2020-12-27 08:06:08 UTC
retroshare depends on <libupnp-1.8.0, and this is blocking cleanup of vulnerable versions of libupnp (<1.14.0). Can anything be done about this?
Comment 1 John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2021-01-31 06:50:41 UTC
Ping Gio. Can you adapt the patch for the version in tree? Is there a new release planned?
Comment 2 Ben Kohler gentoo-dev 2021-02-03 14:11:00 UTC
I see several comments upstream about using miniupnpc instead, eg https://github.com/RetroShare/RetroShare/issues/2072#issuecomment-718154688

That would be an easy ebuild fix.
Comment 3 Larry the Git Cow gentoo-dev 2021-02-03 18:18:05 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3ca02924dcb7e13bc34a84b4ebeea876357ea8be

commit 3ca02924dcb7e13bc34a84b4ebeea876357ea8be
Author:     Ben Kohler <bkohler@gentoo.org>
AuthorDate: 2021-02-03 18:12:20 +0000
Commit:     Ben Kohler <bkohler@gentoo.org>
CommitDate: 2021-02-03 18:18:02 +0000

    net-p2p/retroshare: switch to miniupnpc & fix EAPI=7 conversion
    
    Bug: https://bugs.gentoo.org/761885
    
    Package-Manager: Portage-3.0.14, Repoman-3.0.2
    Signed-off-by: Ben Kohler <bkohler@gentoo.org>

 net-p2p/retroshare/retroshare-0.6.5-r1.ebuild | 153 ++++++++++++++++++++++++++
 1 file changed, 153 insertions(+)
Comment 4 John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2021-02-03 18:20:30 UTC
Thanks Ben! We can use this bug for stabilization of the revbump later.
Comment 5 Gioacchino Mazzurco 2021-02-06 09:18:48 UTC
(In reply to Larry the Git Cow from comment #3)
> The bug has been referenced in the following commit(s):
> 
> https://gitweb.gentoo.org/repo/gentoo.git/commit/
> ?id=3ca02924dcb7e13bc34a84b4ebeea876357ea8be
> 
> commit 3ca02924dcb7e13bc34a84b4ebeea876357ea8be
> Author:     Ben Kohler <bkohler@gentoo.org>
> AuthorDate: 2021-02-03 18:12:20 +0000
> Commit:     Ben Kohler <bkohler@gentoo.org>
> CommitDate: 2021-02-03 18:18:02 +0000
> 
>     net-p2p/retroshare: switch to miniupnpc & fix EAPI=7 conversion
>     
>     Bug: https://bugs.gentoo.org/761885
>     
>     Package-Manager: Portage-3.0.14, Repoman-3.0.2
>     Signed-off-by: Ben Kohler <bkohler@gentoo.org>
> 
>  net-p2p/retroshare/retroshare-0.6.5-r1.ebuild | 153
> ++++++++++++++++++++++++++
>  1 file changed, 153 insertions(+)

Ben are you sure that if libupnp is installed on the system because some other package pull it in, retroshare will really link against miniupnp? Because AFAIR I fixed the upstream code to handle `RS_UPNP_LIB` option correctly at compile time after 0.6.5 was released. But I may remember it wrong.
Comment 6 Ben Kohler gentoo-dev 2021-02-06 18:02:54 UTC
In my testing, it will not attempt to use libupnp as long as  RS_UPNP_LIB=miniupnpc is passed, even if libupnp is installed.

Unfortunately in my first commit of this fix, it was missing \ at the end of the line so the argument wasn't passed properly.

This has been fixed and I'm revbumping to make sure everyone gets the fixed version.
Comment 7 John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2021-03-08 03:12:32 UTC
Ready for stabilization?
Comment 8 Gioacchino Mazzurco 2021-03-08 14:47:39 UTC
(In reply to John Helmert III from comment #7)
> Ready for stabilization?

Good for me!
Comment 9 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-03-11 02:36:00 UTC
amd64 done
Comment 10 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-03-11 13:31:05 UTC
x86 done

all arches done
Comment 11 John Helmert III archtester Gentoo Infrastructure gentoo-dev Security 2021-03-12 13:23:26 UTC
Sorry, please cleanup <0.6.5-r2.
Comment 12 Larry the Git Cow gentoo-dev 2021-03-12 16:01:58 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f234282e99f789bef500cafff2f4bb0458a8807a

commit f234282e99f789bef500cafff2f4bb0458a8807a
Author:     Ben Kohler <bkohler@gentoo.org>
AuthorDate: 2021-03-12 16:01:34 +0000
Commit:     Ben Kohler <bkohler@gentoo.org>
CommitDate: 2021-03-12 16:01:51 +0000

    net-p2p/retroshare: drop old
    
    Closes: https://bugs.gentoo.org/761885
    Package-Manager: Portage-3.0.17, Repoman-3.0.2
    Signed-off-by: Ben Kohler <bkohler@gentoo.org>

 net-p2p/retroshare/retroshare-0.6.5.ebuild | 152 -----------------------------
 1 file changed, 152 deletions(-)