[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor-browser-bundle/master] Backport fix for Bug9665.



commit ab04781bdb01977b9b122681474bfecb96017169
Author: Mike Perry <mikeperry-git@xxxxxxxxxxxxxx>
Date:   Mon Apr 7 20:05:47 2014 -0700

    Backport fix for Bug9665.
---
 gitian/descriptors/linux/gitian-tor.yml   |    4 +-
 gitian/descriptors/mac/gitian-tor.yml     |    2 +
 gitian/descriptors/windows/gitian-tor.yml |    2 +
 gitian/patches/bug9665.patch              |  108 +++++++++++++++++++++++++++++
 4 files changed, 115 insertions(+), 1 deletion(-)

diff --git a/gitian/descriptors/linux/gitian-tor.yml b/gitian/descriptors/linux/gitian-tor.yml
index 698fcba..15ba469 100644
--- a/gitian/descriptors/linux/gitian-tor.yml
+++ b/gitian/descriptors/linux/gitian-tor.yml
@@ -28,8 +28,9 @@ files:
 - "bug5018.patch"
 - "bug9229.patch"
 - "bug11156.patch"
-- "bug11069.patch"
 - "bug11200.patch"
+- "bug11069.patch"
+- "bug9665.patch"
 - "dzip.sh"
 - "openssl.tar.gz"
 script: |
@@ -98,6 +99,7 @@ script: |
       git am ~/build/bug11156.patch
     fi
     git am ~/build/bug11200.patch
+    git am ~/build/bug9665.patch
   fi
   mkdir -p $OUTDIR/src
   #git archive HEAD | tar -x -C $OUTDIR/src
diff --git a/gitian/descriptors/mac/gitian-tor.yml b/gitian/descriptors/mac/gitian-tor.yml
index 30c23bb..5ea3b4e 100644
--- a/gitian/descriptors/mac/gitian-tor.yml
+++ b/gitian/descriptors/mac/gitian-tor.yml
@@ -31,6 +31,7 @@ files:
 - "bug11156.patch"
 - "bug11200.patch"
 - "bug11069.patch"
+- "bug9665.patch"
 - "apple-uni-sdk-10.6_20110407-0.flosoft1_i386.deb"
 - "multiarch-darwin11-cctools127.2-gcc42-5666.3-llvmgcc42-2336.1-Linux-120724.tar.xz"
 - "dzip.sh"
@@ -109,6 +110,7 @@ script: |
       git am ~/build/bug11156.patch
     fi
     git am ~/build/bug11200.patch
+    git am ~/build/bug9665.patch
   fi
   mkdir -p $OUTDIR/src
   git archive HEAD | tar -x -C $OUTDIR/src
diff --git a/gitian/descriptors/windows/gitian-tor.yml b/gitian/descriptors/windows/gitian-tor.yml
index 1698726..6369e7f 100644
--- a/gitian/descriptors/windows/gitian-tor.yml
+++ b/gitian/descriptors/windows/gitian-tor.yml
@@ -30,6 +30,7 @@ files:
 - "bug11156.patch"
 - "bug11200.patch"
 - "bug11069.patch"
+- "bug9665.patch"
 - "binutils.tar.bz2"
 - "dzip.sh"
 - "openssl.tar.gz"
@@ -105,6 +106,7 @@ script: |
       git am ~/build/bug11156.patch
     fi
     git am ~/build/bug11200.patch
+    git am ~/build/bug9665.patch
   fi
   mkdir -p $OUTDIR/src
   git archive HEAD | tar -x -C $OUTDIR/src
diff --git a/gitian/patches/bug9665.patch b/gitian/patches/bug9665.patch
new file mode 100644
index 0000000..aeab37b
--- /dev/null
+++ b/gitian/patches/bug9665.patch
@@ -0,0 +1,108 @@
+From 08ae53e400ff6fa2d8147aad440c38173c106cae Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?F=C3=A1bio=20J=2E=20Bertinatto?= <fabiojrb@xxxxxxxxx>
+Date: Tue, 5 Nov 2013 00:50:16 -0200
+Subject: [PATCH 1/3] Fix bug9665
+
+---
+ src/or/connection_or.c | 5 +++++
+ 1 file changed, 5 insertions(+)
+
+diff --git a/src/or/connection_or.c b/src/or/connection_or.c
+index 04ad2cc..ba3ac00 100644
+--- a/src/or/connection_or.c
++++ b/src/or/connection_or.c
+@@ -1195,6 +1195,11 @@ connection_or_connect(const tor_addr_t *_addr, uint16_t port,
+                "your pluggable transport proxy stopped running.",
+                fmt_addrport(&TO_CONN(conn)->addr, TO_CONN(conn)->port),
+                transport_name, transport_name);
++
++      control_event_bootstrap_problem(
++                                "Can't connect to bridge",
++                                END_OR_CONN_REASON_NO_ROUTE);
++
+     } else {
+       log_warn(LD_GENERAL, "Tried to connect to '%s' through a proxy, but "
+                "the proxy address could not be found.",
+-- 
+1.8.1.2
+
+From 754a50592c412d95d2eb48038784d0ef725a7dc2 Mon Sep 17 00:00:00 2001
+From: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
+Date: Mon, 7 Apr 2014 13:41:07 -0400
+Subject: [PATCH 2/3] Forward-port bug9665 fix to work with our fix for 11069
+
+---
+ src/or/connection_or.c | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/src/or/connection_or.c b/src/or/connection_or.c
+index ba3ac00..01ff4dc 100644
+--- a/src/or/connection_or.c
++++ b/src/or/connection_or.c
+@@ -1198,7 +1198,8 @@ connection_or_connect(const tor_addr_t *_addr, uint16_t port,
+ 
+       control_event_bootstrap_problem(
+                                 "Can't connect to bridge",
+-                                END_OR_CONN_REASON_NO_ROUTE);
++                                END_OR_CONN_REASON_NO_ROUTE,
++                                conn);
+ 
+     } else {
+       log_warn(LD_GENERAL, "Tried to connect to '%s' through a proxy, but "
+-- 
+1.8.1.2
+
+From 90341b4852bf88f1fdf9fd150fa2f5c47f88b2cb Mon Sep 17 00:00:00 2001
+From: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
+Date: Mon, 7 Apr 2014 13:44:22 -0400
+Subject: [PATCH 3/3] For missing transport, say "PT_MISSING" not "NO_ROUTE"
+
+---
+ src/or/connection_or.c | 2 +-
+ src/or/or.h            | 3 ++-
+ src/or/reasons.c       | 2 ++
+ 3 files changed, 5 insertions(+), 2 deletions(-)
+
+diff --git a/src/or/connection_or.c b/src/or/connection_or.c
+index 01ff4dc..6572a91 100644
+--- a/src/or/connection_or.c
++++ b/src/or/connection_or.c
+@@ -1198,7 +1198,7 @@ connection_or_connect(const tor_addr_t *_addr, uint16_t port,
+ 
+       control_event_bootstrap_problem(
+                                 "Can't connect to bridge",
+-                                END_OR_CONN_REASON_NO_ROUTE,
++                                END_OR_CONN_REASON_PT_MISSING,
+                                 conn);
+ 
+     } else {
+diff --git a/src/or/or.h b/src/or/or.h
+index 38ab176..1b35c1f 100644
+--- a/src/or/or.h
++++ b/src/or/or.h
+@@ -604,7 +604,8 @@ typedef enum {
+ #define END_OR_CONN_REASON_NO_ROUTE       6 /* no route to host/net */
+ #define END_OR_CONN_REASON_IO_ERROR       7 /* read/write error */
+ #define END_OR_CONN_REASON_RESOURCE_LIMIT 8 /* sockets, buffers, etc */
+-#define END_OR_CONN_REASON_MISC           9
++#define END_OR_CONN_REASON_PT_MISSING     9 /* PT failed or not available */
++#define END_OR_CONN_REASON_MISC           10
+ 
+ /* Reasons why we (or a remote OR) might close a stream. See tor-spec.txt for
+  * documentation of these.  The values must match. */
+diff --git a/src/or/reasons.c b/src/or/reasons.c
+index 0674474..750e89b 100644
+--- a/src/or/reasons.c
++++ b/src/or/reasons.c
+@@ -231,6 +231,8 @@ orconn_end_reason_to_control_string(int r)
+       return "RESOURCELIMIT";
+     case END_OR_CONN_REASON_MISC:
+       return "MISC";
++    case END_OR_CONN_REASON_PT_MISSING:
++      return "PT_MISSING";
+     case 0:
+       return "";
+     default:
+-- 
+1.8.1.2
+



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits