Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 588944

Summary: sys-apps/portage: make doins error out if insinto wasn't used
Product: Portage Development Reporter: Michał Górny <mgorny>
Component: CoreAssignee: Portage team <dev-portage>
Status: CONFIRMED ---    
Severity: normal CC: mjo, pms
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 627094, 627096, 627098    
Bug Blocks: 563798    

Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-07-16 18:35:36 UTC
Proxy-maint just hit an ebuild where proxied maintainer used 'doins' to install files recursively into /. Aside the fact that this caused wrong permissions, I think that PMS doesn't specify any default path for doins.

Please make Portage error out if 'doins' is called with no path set by earlier 'insinto' call.
Comment 1 Michael Orlitzky gentoo-dev 2016-07-16 20:44:59 UTC
Here's a list of current offenders (probably not comprehensive):

dev-util/intel-ocl-sdk-2.0.31360
media-gfx/evoluspencil-2.0.5
sci-physics/geant-data-4.10.00
sci-physics/geant-data-4.10.01
sci-physics/geant-data-4.9.6_p02
sci-electronics/systemc-2.2.0-r2
sci-libs/spqr-1.2.3-r1
sci-visualization/xyscan-3.3.1

# Relies on the doins() call in linux-mod_src_install().
sys-fs/aufs4-0_pre20160627
sys-fs/aufs4-0_pre20160219
sys-fs/aufs4-0_pre20160223
sys-fs/aufs3-3_p20160627
sys-fs/aufs3-3_p20160223
sys-fs/aufs3-3_p20160219

# Bug, should be using dodoc() instead of doins().
dev-libs/gecode-3.7.3
Comment 2 Zac Medico gentoo-dev 2016-07-16 21:17:19 UTC
This looks like a good candidate for an eqawarn QA Notice.