tracker_icons.py does not clean up /tmp entries from failed hosts

[rexar]

If Deluge attempts to obtain a tracker icon for a tracker that is unreachable, or has a re-direct to self (perpetual HTTP 301/302) the errorback routines in tracker_icons.py do not have access to the temporary filehandles / paths created as part of the download_page method, and as such is unable to clean up these pre-allocated resources.

Over a period of time under Linux, this fills the /tmp directory with a large amount of junk files in the form of deluge_ticon.xxxxx

Attached is a workaround targetting the 1.3.15 release version of tracker_icons.py which allows the errorback routines to clean files allocated from failed attempts.

This was developed against my scenario which is the GTK client connecting to a remote server, it hasn't been tested for other use cases.

[rexar]

Patch file against 1.3.15 release version of tracker_icons.py

[Calum]

Missed seeing this bug, should have been fixed, thanks for the workaround patch.

Duplicate reported in #3519

[Calum]

Fixed via [68c75ccc0540b435ade9a833ba5eadd5f0464f5f]