[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218582035 Thanks @oknet --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this f

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/625 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218544819 Build finished successfully. No test results found. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218541607 Build triggered. sha1 is merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project doe

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218540994 [approve ci] --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feat

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218525556 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pro

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-10 Thread oknet
Github user oknet commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218362265 int UnixNetVConnection::connectUp(EThread *t, int fd) { int res; thread = t; if (check_net_throttle(CONNECT, submit_time)) { ch

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-10 Thread bryancall
Github user bryancall commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218203477 How are you seeing the double free? The original code looks good. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-10 Thread oknet
GitHub user oknet opened a pull request: https://github.com/apache/trafficserver/pull/625 TS-4432: TSVConnFdCreate, double free on connectUp failure You can merge this pull request into a Git repository by running: $ git pull https://github.com/oknet/trafficserver patch-9 Alt