codex-storage/deluge
1
0
Fork 0
mirror of https://github.com/codex-storage/deluge.git synced 2025-01-27 11:45:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

[#2335] [GTKUI] Fix startup failing with 'cannot acquire lock'

Browse Source 
This issue was caused by an unclean shutdown of Deluge, usually on system shutdown, and upon rebooting
the PID stored in the lockfile is in used by another process thus the lockfile is never removed. It
affects users with Deluge set in startup applications as the PIDs are more likely to be reused.

 * Lockfile is removed if Deluge is restarted in IPC.
 * Renamed the old_tempfile variable to make it clearer as to it's role.
This commit is contained in:
Calum Lind 2014-10-02 17:15:00 +01:00
parent 11c6e387d5
commit d805f99534
1 changed files with 15 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
deluge/ui/gtkui/ipcinterface.py
View File

@ -99,24 +99,28 @@ class IPCInterface(component.Component):
sys.exit(0)
else:
# Find and remove any restart tempfiles
old_tempfile = glob(os.path.join(ipc_dir, 'tmp*deluge'))
for f in old_tempfile:
restart_tempfile = glob(os.path.join(ipc_dir, 'tmp*deluge'))
for f in restart_tempfile:
os.remove(f)
lockfile = socket + ".lock"
log.debug("Checking if lockfile exists: %s", lockfile)
if os.path.lexists(lockfile) or os.path.lexists(socket):
try:
os.kill(int(os.readlink(lockfile)), 0)
except OSError:
if os.path.lexists(lockfile):
def delete_lockfile():
log.debug("Removing lockfile since it's stale.")
try:
os.remove(lockfile)
except OSError as ex:
log.error("Failed to delete IPC lockfile file: %s", ex)
try:
os.remove(socket)
except OSError as ex:
log.error("Failed to delete IPC socket file: %s", ex)
log.error("Failed to delete lockfile: %s", ex)
try:
os.kill(int(os.readlink(lockfile)), 0)
except OSError:
delete_lockfile()
else:
if restart_tempfile:
log.warning("Found running PID but it is not a Deluge process, removing lockfile...")
delete_lockfile()
try:
self.factory = Factory()
self.factory.protocol = IPCProtocolServer
@ -133,7 +137,7 @@ class IPCInterface(component.Component):
gtk.gdk.notify_startup_complete()
sys.exit(0)
else:
if old_tempfile:
if restart_tempfile:
log.error("Deluge restart failed: %s", ex)
sys.exit(1)
else: