From 72bdf0f920380807b1da4bb5be3523e65f618182 Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Fri, 5 Aug 2005 07:58:59 +0000 Subject: Bug 288170: importxml.pl and move.pl should not write to $datadir/maillog - Patch by Olav Vitters r=LpSolit a=justdave --- move.pl | 36 ------------------------------------ 1 file changed, 36 deletions(-) (limited to 'move.pl') diff --git a/move.pl b/move.pl index 030d03ac0..92a051b63 100755 --- a/move.pl +++ b/move.pl @@ -36,8 +36,6 @@ use Bugzilla::Bug; use Bugzilla::Config qw(:DEFAULT $datadir); use Bugzilla::BugMail; -$::lockcount = 0; - unless ( Param("move-enabled") ) { print "\n

Sorry. Bug moving is not enabled here. "; print "If you need to move a bug, contact " . Param("maintainer"); @@ -48,37 +46,6 @@ Bugzilla->login(LOGIN_REQUIRED); my $cgi = Bugzilla->cgi; -sub Log { - my ($str) = (@_); - Lock(); - open(FID, ">>$datadir/maillog") || die "Can't write to $datadir/maillog"; - print FID time2str("%D %H:%M", time()) . ": $str\n"; - close FID; - Unlock(); -} - -sub Lock { - if ($::lockcount <= 0) { - $::lockcount = 0; - open(LOCKFID, ">>$datadir/maillock") || die "Can't open $datadir/maillock: $!"; - my $val = flock(LOCKFID,2); - if (!$val) { # '2' is magic 'exclusive lock' const. - print $cgi->header(); - print "Lock failed: $val\n"; - } - chmod 0666, "$datadir/maillock"; - } - $::lockcount++; -} - -sub Unlock { - $::lockcount--; - if ($::lockcount <= 0) { - flock(LOCKFID,8); # '8' is magic 'unlock' const. - close LOCKFID; - } -} - if (!defined $cgi->param('buglist')) { print $cgi->header(); $template->put_header("Move Bugs"); @@ -171,6 +138,3 @@ $template->process("bug/show.xml.tmpl", { bugs => \@bugs, $msg .= "\n"; Bugzilla::BugMail::MessageToMTA($msg); - -my $logstr = "XML: bugs $buglist sent to $to"; -Log($logstr); -- cgit v1.2.3-24-g4f1b