File transfers are never canceled for real

[Edtech]

If you cancel a transfert, the file is not released and do not can be removed as long as Gajim was not closed.

[js]

I had a look at this, Asterix. It's not a Windows bug, it's a deeper bug - Windows just showed it because it can't delete opened files.

Gajim never closes the file handle when you cancel a transfer. In fact, it doesn't even cancel it properly. That explains why someone sometimes got a file that I already canceled. Everything that Gajim does it remove it from the FT window and send a rejection stanza, nothing more.

If you cancel the file before the other side accepts, the other side is still able to get the whole file. What we do is: We send a rejection and rely on the other side to close the stream. But that never closes our file handle. And if the other side doesn't close, we still transfer.

We need to rewrite cancelling file transfers, as this doesn't work at all.

I had a look at it, but I think the whole file transfer code is a bit strange - GUI and actual file transfer code is happily mixed. Could you maybe fix this, Asterix? You can also see on Linux that the file (and the stream, if the other side doesn't close it!) is never ended using lsof.

[js]

Asterix, this is for 0.12 IMO, as this is a serious bug. It may leak private information, for example if you drop a file by accident in the chat window. Now you cancel it, but the other side still receives it - VERY, VERY bad. Already happened to me.

[asterix]

this is in Gajim since ages, and nat many ppl complains. Ok it's bad, but it's hard to fix, and I don't want to release in 2 monthes. As currently all dev seems to be busy, I won't fix that for 0.12.

[cleese]

please fix this :)

[Yann Leboulanger <asterix@…>]

(In [04270cbd7e04]) don't send a canceld filetransfer. Fixes #3586