Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 507248 - net-misc/openssh: /var/empty as privilege separation directory is a bad choice
Summary: net-misc/openssh: /var/empty as privilege separation directory is a bad choice
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-09 15:11 UTC by spacefrogg-gentoo
Modified: 2015-07-08 09:08 UTC (History)
1 user (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 spacefrogg-gentoo 2014-04-09 15:11:49 UTC
I do not know if /var/empty is picked as the privilege separation directory for openssh for a reason. But I cannot imagine a good one. The directory is called, well, 'empty' for a reason. It should be left empty. On some BSD's for instance it is actually immutable to make sure it stays empty, so it is on my machines.
I suggest to pick another directory such as /var/{lib,run}/openssh to carry the live data. I fail to see why openssh, in particular, should have the right to highjack /var/empty for its purposes as other programs might well depend on the emptiness properties of this directory.

Reproducible: Always
Comment 1 SpanKY gentoo-dev 2014-04-09 19:34:58 UTC
what "live" data are you talking about exactly ?  the only thing in there is /dev/ and that's when USE=hpn is enabled, and we can probably drop that.
Comment 2 spacefrogg-gentoo 2014-04-10 10:13:19 UTC
I don't know what live data openssh needs, but at least the hpn patch introduces a directory in /var/empty and portage also introduces a .keep file. And this kind of contradicts the meaning of '/var/empty'.
Comment 3 SpanKY gentoo-dev 2015-07-08 09:08:11 UTC
starting with openssh-6.9_p1-r2, openssh no longer carries the server logging patch, which means the modifications to /var/empty are no longer needed