From 155c93bb935807b0a77a4fcfa989c989b7980ae6 Mon Sep 17 00:00:00 2001 From: Thierry Vignaud Date: Fri, 7 Sep 2012 17:45:16 +0000 Subject: (confirm_upgrade) check that choosen download directory is owned by root (mga#6083) --- mgaapplet | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'mgaapplet') diff --git a/mgaapplet b/mgaapplet index ca57ffb5..959f9edb 100755 --- a/mgaapplet +++ b/mgaapplet @@ -395,6 +395,10 @@ sub confirm_upgrade() { Gtk2::FileChooserButton->new(N("Browse"), 'select-folder'), 'current-folder-changed' => sub { $temp_dir = $_[0]->get_current_folder; + my $ok = 1; + require urpm; + urpm::check_cache_dir({ fatal => sub { $ok = 0 } }, $temp_dir); + $ok or ask_warn(N("Error"), N("You must choose a directory owned by the super administrator!")); }); $browse->set_current_folder($download_dir); @@ -426,6 +430,10 @@ sub confirm_upgrade() { setVar('DO_NOT_ASK_FOR_DISTRO_UPGRADE', bool2text($warn_me)); $local_config{DO_NOT_ASK_FOR_DISTRO_UPGRADE} = bool2text($warn_me); if ($res) { + require urpm; + my $ok = 1; + urpm::check_cache_dir({ fatal => sub { $ok = 0 } }, $temp_dir); + $ok or goto &confirm_upgrade; $download_dir = $temp_dir; really_confirm_upgrade(); } else { -- cgit v1.2.1