tor->isRunning not always saved to .resume file

[dairiki]

When a torrent is stopped (via tr_torrentStop()) the dirty flag (tor->isDirty) does not get set. This means that the paused flag in the .resume file for the torrent will still be clear. If transmission is stopped and restarted, the torrent will resume to a running state (it should remain stopped.)

A (very small) patch is attached.

I've only tested this with transmission-daemon, but from looking at the code, I'd guess its a more general libtransmission problem.

---

Recipe to reproduce bug

Have transmission-daemon running and seeding a torrent.

(Look at the .resume file for the torrent and verify that paused is set to 0.)

Use transmission-remote to stop the torrent.

At this point you can either shut down transmission-daemon, or wait two minutes for its dirty-resume-save timer to time out. Then check the .resume file for the torrent. Note that paused is till set to 0. (Since the torrent is paused, it should be set to 1.)

(If, after stopping the torrent (with transmission-remote) one then alters some other property of the torrent (say the torrents seedratio), the .resume file will be marked dirty, and (after two minutes or shutting down the daemon) paused will be correctly set to 1 in the .resume file.)

[dairiki]

Patch on svn r9339

[charles]

Thanks for the patch!

Added to trunk in r9340 for 1.80

[dairiki]

Cool! Thanks for the quick application. (And thanks for transmission in general!)

[charles]

backported to 1.7x branch in r9871