diff options
author | Max Kellermann <max@duempel.org> | 2014-12-14 22:48:46 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2014-12-14 22:49:16 +0100 |
commit | 02563a35f0ce4bb5154a68bd568d0263c76ec261 (patch) | |
tree | 51eadcf645c6625cfffb355729f420e675291ae2 /src/lib/nfs/Connection.cxx | |
parent | d653f35bb7488b2da3412fc23e16b4da1770f8d4 (diff) | |
download | mpd-02563a35f0ce4bb5154a68bd568d0263c76ec261.tar.gz mpd-02563a35f0ce4bb5154a68bd568d0263c76ec261.tar.xz mpd-02563a35f0ce4bb5154a68bd568d0263c76ec261.zip |
lib/nfs/Connection: fix reconnect after mount failure
When mounting had not yet finished, SocketMonitor::IsDefined() was
always false, due to the workaround at the beginning of the function
that calls SocketMonitor::Steal(). This commit drops the IsDefined()
check because it was never necessary and breaks reconnect.
Diffstat (limited to 'src/lib/nfs/Connection.cxx')
-rw-r--r-- | src/lib/nfs/Connection.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/nfs/Connection.cxx b/src/lib/nfs/Connection.cxx index e0d8ad774..04c5a9e53 100644 --- a/src/lib/nfs/Connection.cxx +++ b/src/lib/nfs/Connection.cxx @@ -483,7 +483,7 @@ NfsConnection::OnSocketReady(unsigned flags) DestroyContext(); closed = true; - } else if (SocketMonitor::IsDefined() && nfs_get_fd(context) < 0) { + } else if (nfs_get_fd(context) < 0) { /* this happens when rpc_reconnect_requeue() is called after the connection broke, but autoreconnect was disabled - nfs_service() returns 0 */ |