Comment 2 for bug 904033

Revision history for this message
Harri Porten (porten-deactivatedaccount) wrote :

The nasty part about the code (not necessarily the patch itself) is that 'false' will be returned from release() in cases of both file missing *and* the release failing.

Maybe it would be easiest to reverse the logic? "Failed to remove lock file" is printed out if the operation fails but *no* output is printed in the two other cases.