[ruby-core:69577] [Ruby trunk - Bug #11260] [Open] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[ruby-core:69577] [Ruby trunk - Bug #11260] [Open] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[eregontp]

Issue #11260 has been reported by Benoit Daloze.

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260

* Author: Benoit Daloze
* Status: Open
* Priority: Normal
* Assignee: Shugo Maeda
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:69578] [Ruby trunk - Bug #11260] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[eregontp]

Issue #11260 has been updated by Benoit Daloze.


@shugo, can I commit this?

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-52917

* Author: Benoit Daloze
* Status: Open
* Priority: Normal
* Assignee: Shugo Maeda
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:69753] [Ruby trunk - Bug #11260] [Assigned] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[shugo]

Issue #11260 has been updated by Shugo Maeda.

Status changed from Open to Assigned
Assignee changed from Shugo Maeda to Benoit Daloze

Benoit Daloze wrote:
> @shugo, can I commit this?

The patch looks fine.  Please commit it.

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-53130

* Author: Benoit Daloze
* Status: Assigned
* Priority: Normal
* Assignee: Benoit Daloze
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:69755] [Ruby trunk - Bug #11260] [Closed] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[eregontp]

Issue #11260 has been updated by Benoit Daloze.

Status changed from Assigned to Closed

Thanks, committed in r51046.

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-53132

* Author: Benoit Daloze
* Status: Closed
* Priority: Normal
* Assignee: Benoit Daloze
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:69786] [Ruby trunk - Bug #11260] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[usa]

Issue #11260 has been updated by Usaku NAKAMURA.

Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN to 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: REQUIRED

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-53164

* Author: Benoit Daloze
* Status: Closed
* Priority: Normal
* Assignee: Benoit Daloze
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: REQUIRED
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:70354] [Ruby trunk - Bug #11260] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[nagachika00]

Issue #11260 has been updated by Tomoyuki Chikanaga.

Backport changed from 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: REQUIRED to 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: DONE

Backported into `ruby_2_2` branch at r51556.

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-53770

* Author: Benoit Daloze
* Status: Closed
* Priority: Normal
* Assignee: Benoit Daloze
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: DONE
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/

[ruby-core:70417] [Ruby trunk - Bug #11260] [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails

[usa]

Issue #11260 has been updated by Usaku NAKAMURA.

Backport changed from 2.0.0: WONTFIX, 2.1: REQUIRED, 2.2: DONE to 2.0.0: WONTFIX, 2.1: DONE, 2.2: DONE

ruby_2_1 r51610 merged revision(s) 51046.

----------------------------------------
Bug #11260: [PATCH] Net::FTP keeps a TCPServer opened when sending the data port fails
https://bugs.ruby-lang.org/issues/11260#change-53825

* Author: Benoit Daloze
* Status: Closed
* Priority: Normal
* Assignee: Benoit Daloze
* ruby -v: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
* Backport: 2.0.0: WONTFIX, 2.1: DONE, 2.2: DONE
----------------------------------------
`Net::FTP` will keep the `TCPServer` created in `makeport` opened if `sendport` fails.
The patch sends the port in its (only) caller instead, so the TCPServer is appropriately closed no matter the outcome.

This was detected with RubySpec and a leak checker based on test/lib/leakchecker.rb.

---Files--------------------------------
0001-lib-net-ftp.rb-makeport-close-the-TCPServer.patch (3.43 KB)


-- 
https://bugs.ruby-lang.org/