byterazor/qpsmtpd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove unnecessary calls to cleanup_and_return()

Browse Source 
prune_db() and migrate_db() already call unlock(), calling
cleanup_and_return() only added a second (redundant) call to unlock()
This commit is contained in:
Jared Johnson 2015-01-21 17:31:47 -06:00
parent b97508b217
commit 8c1c5c851d
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
plugins/greylisting
View File

@ -437,7 +437,7 @@ sub convert_db {
} }
$self->db->unlock; $self->db->unlock;
$self->log(LOGINFO, "converted $converted of $count DB entries"); $self->log(LOGINFO, "converted $converted of $count DB entries");
return $self->cleanup_and_return(DECLINED); return DECLINED;
} }
sub prune_db { sub prune_db {
@ -456,7 +456,7 @@ sub prune_db {
} }
$self->db->unlock; $self->db->unlock;
$self->log(LOGINFO, "pruned $pruned of $count DB entries"); $self->log(LOGINFO, "pruned $pruned of $count DB entries");
return $self->cleanup_and_return(DECLINED); return DECLINED;
} }
sub exclude { sub exclude {