Skip to content

Commit

Permalink
fix the 'releaselock' command to not crash the server. introduced
Browse files Browse the repository at this point in the history
when jonathan did pluggable locking backends.  fortunately we never
actually use the release command.  but we will....

fix the client to use persistent connections to lock servers, caching
them in the parent client objects.  so lock objects now have up
references to their client, and it asks for sockets.  when give one,
it caches that it has a lock on "10.0.0.34:34" only, not the actual
socket, and gets it back later on DESTROY to unlock it.

this will waste less ephemeral ports and syndication will catch back up.




git-svn-id: http://code.sixapart.com/svn/ddlockd/trunk@33 75bc8fc0-0210-0410-8e5e-a32055405bc5
  • Loading branch information
bradfitz committed Aug 29, 2006
1 parent 5621219 commit 6d3c0b7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ddlockd
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ sub cmd_releaselock {
return $self->err_line("empty_lock") unless length($lock);
return $self->err_line("didnthave") unless $self->{locks}{$lock};

_release_lock($self, $lock);
$self->_release_lock($lock);
return $self->ok_line;
}

Expand Down

0 comments on commit 6d3c0b7

Please sign in to comment.