PortMapping Bug report.

[vinny]

Hi!

Sorry for my English

If PortMapping disabled, function tr_sessionClose close arbitrary file descriptor (usually zero).

Workaround:

Code: Select all

--- a/userapps/opensource/transmission-1.74/libtransmission/natpmp.c    Tue Oct 20 13:02:46 2009 +0300
+++ b/userapps/opensource/transmission-1.74/libtransmission/natpmp.c    Sun Oct 25 23:51:19 2009 +0200
@@ -89,7 +89,7 @@
 {
     if( nat )
     {
-        closenatpmp( &nat->natpmp );
+        if (nat->state != TR_NATPMP_DISCOVER) closenatpmp (&nat->natpmp);
         tr_free( nat );
     }
 }

[Jordan]

Thanks for this bug report!

I've opened a ticket for this <http://trac.transmissionbt.com/ticket/2534> for tracking purposes, and committed a fix to trunk. I probably wouldn't've seen this issue if you hadn't reported it, so thanks again...