DBUpgrade2: verhindern, dass $::form->get_standard_dbh eine zweite Verbindung öffnet
authorMoritz Bunkus <m.bunkus@linet-services.de>
Mon, 15 Jul 2013 10:32:07 +0000 (12:32 +0200)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Mon, 15 Jul 2013 10:36:47 +0000 (12:36 +0200)
Ansonsten sind Deadlocks absolut möglich.

SL/DBUpgrade2.pm
SL/Form.pm

index eba336f..3cdb9a4 100644 (file)
@@ -243,6 +243,7 @@ sub process_perl_script {
   $dbh->begin_work;
 
   # setup dbup_ export vars & run script
+  my $old_dbh       = $::form->set_standard_dbh($dbh);
   my %dbup_myconfig = map { ($_ => $::form->{$_}) } qw(dbname dbuser dbpasswd dbhost dbport dbconnect);
   my $result        = eval {
     SL::DBUpgrade2::Base::execute_script(
@@ -255,6 +256,8 @@ sub process_perl_script {
 
   my $error = $EVAL_ERROR;
 
+  $::form->set_standard_dbh($old_dbh);
+
   $dbh->rollback if 1 != ($result // -1);
 
   return $error if $self->{return_on_error} && (1 != ($result // -1));
index 79ff703..b18bc67 100644 (file)
@@ -1403,6 +1403,14 @@ sub get_standard_dbh {
   return $standard_dbh;
 }
 
+sub set_standard_dbh {
+  my ($self, $dbh) = @_;
+  my $old_dbh      = $standard_dbh;
+  $standard_dbh    = $dbh;
+
+  return $old_dbh;
+}
+
 sub date_closed {
   $main::lxdebug->enter_sub();