xenbits.xensource.com Git - libvirt.git/commit

]> xenbits.xensource.com Git - libvirt.git/commit

author	Daniel P. Berrange <berrange@redhat.com>
	Wed, 9 Oct 2013 10:18:15 +0000 (11:18 +0100)
committer	Daniel P. Berrange <berrange@redhat.com>
	Mon, 21 Oct 2013 13:03:52 +0000 (14:03 +0100)
commit	1e4a02bdfe6307f93763fa2c9681f280c564aee5
tree	da11dc1d0726743c2581adab11411009f7eeb979	tree
parent	9b8f307c6ad002a17a0510513883d06395636793	commit \| diff

Remove all direct use of getenv

Unconditional use of getenv is not secure in setuid env.
While not all libvirt code runs in a setuid env (since
much of it only exists inside libvirtd) this is not always
clear to developers. So make all the code paranoid, even
if it only ever runs inside libvirtd.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

17 files changed:

daemon/libvirtd.c		diff \| blob \| blame \| history
src/driver.c		diff \| blob \| blame \| history
src/libvirt.c		diff \| blob \| blame \| history
src/locking/lock_daemon.c		diff \| blob \| blame \| history
src/locking/lock_driver_lockd.c		diff \| blob \| blame \| history
src/locking/lock_manager.c		diff \| blob \| blame \| history
src/lxc/lxc_driver.c		diff \| blob \| blame \| history
src/remote/remote_driver.c		diff \| blob \| blame \| history
src/rpc/virnettlscontext.c		diff \| blob \| blame \| history
src/util/virauth.c		diff \| blob \| blame \| history
src/util/virfile.c		diff \| blob \| blame \| history
src/util/virlog.c		diff \| blob \| blame \| history
src/util/virrandom.c		diff \| blob \| blame \| history
src/util/virutil.c		diff \| blob \| blame \| history
src/vbox/vbox_XPCOMCGlue.c		diff \| blob \| blame \| history
src/vbox/vbox_tmpl.c		diff \| blob \| blame \| history
tools/virsh.c		diff \| blob \| blame \| history

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom