GNU bug report logs - #62991
30.0.50; normal-backup-enable-predicate does not consider /dev/shm

Previous Next

Full log

Message #8 received at 62991 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Vasilij Schneidermann <mail <at> vasilij.de>
Cc: 62991 <at> debbugs.gnu.org
Subject: Re: bug#62991: 30.0.50;
 normal-backup-enable-predicate does not consider /dev/shm
Date: Fri, 21 Apr 2023 14:56:18 +0300

> Date: Fri, 21 Apr 2023 12:49:01 +0200
> From: Vasilij Schneidermann <mail <at> vasilij.de>
> 
> I've found that the `pass` password-store creates temporary files under
> /dev/shm rather than /tmp and editing those files with emacs creates
> backup files (which may leak passwords outside that directory if
> `backup-directory-alist` is customized to place them elsewhere)`. This
> seems to be due to `normal-backup-enable-predicate` specifically testing
> for /tmp (and two other temporary directories), but not other
> directories backed by a tmpfs file system (such as /dev/shm).  There
> does not appear to be a customizable to extend the list of directories
> it's checking for either. One more candidate for exclusion is the
> /run/user/<uid> directory (it's backed by tmpfs as well, but has
> different uses).

We could indeed teach normal-backup-enable-predicate about /dev/shm on
systems where that is appropriate, but I'm curious why you say this is
not customizable: normal-backup-enable-predicate is called via a
variable, backup-enable-predicate, so you could customize that by
writing your own predicate function, and assigning
backup-enable-predicate to that function.  Or what am I missing?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.