chiark / gitweb /
stop passing passphrases via args, instead use prepared files
authorHans-Christoph Steiner <hans@eds.org>
Tue, 1 Apr 2014 01:02:42 +0000 (21:02 -0400)
committerHans-Christoph Steiner <hans@eds.org>
Tue, 1 Apr 2014 01:02:42 +0000 (21:02 -0400)
commit525759b235f20f5f21dc8af5d5e7626a68639bfe
tree65588a29d8c0689b955e8ed035d59e0e467acfde
parentcaa88ec38838a3c1a4a71834cd65a101e378cf1e
stop passing passphrases via args, instead use prepared files

Any process can read the process table, and can therefore see the entire
command line of any other process.  That means its a bad idea to ever put
passwords as part of a command line.  Python is executing keytool and
jarsigner command lines here, so now instead of putting the password on the
command line, a file is passed instead with suitable file permissions.
This should reduce the exposure a lot.  But still, sensitive passwords
should not be written to any text file.

This change requires OpenJDK-7 since the :file option to -storepass and
-keypass was only added in Java 7's keytool and jarsigner.
fdroidserver/common.py
fdroidserver/init.py
fdroidserver/publish.py
fdroidserver/update.py
sampleconfigs/config.py