Bug 1636 - Minor Bug in BayesStore.pm
Summary: Minor Bug in BayesStore.pm
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamassassin (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P5 normal
Target Milestone: 2.50
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords: backport
Depends on:
Blocks:
 
Reported: 2003-03-14 07:09 UTC by Eugene Miretsky
Modified: 2003-03-18 09:39 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
A patch to fix this problem patch None Eugene Miretsky [NoCLA]
Sorry, disregard prev patch -- incorrect file order... patch None Eugene Miretsky [NoCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Miretsky 2003-03-14 07:09:57 UTC 
Found minor bug in BayesStore.pm: lines 786 - 789:
  my $failure;
  if ($@) {             # if we died, untie the dbs.
    my $failure = $@;
  }

failure variable is redeclared inside if statement --> error message is not saved
correctly
Comment 1 Eugene Miretsky 2003-03-14 07:11:20 UTC 
Created attachment 746 [details]
A patch to fix this problem
Comment 2 Eugene Miretsky 2003-03-14 07:12:20 UTC 
Created attachment 747 [details]
Sorry, disregard prev patch -- incorrect file order...
Comment 3 Justin Mason 2003-03-18 16:24:46 UTC 
another one missed.

OKAY: by me
Comment 4 Theo Van Dinter 2003-03-18 16:42:10 UTC 
OKAY: here too
Comment 5 Justin Mason 2003-03-18 18:33:03 UTC 
OK, applied to HEAD; someone else had already applied it to b2_5_0 somehow... wierd
Comment 6 Justin Mason 2003-03-18 18:39:41 UTC 
ok, my checkout was bad, now fixed and patch applied.