- tor-commits - lists.torproject.org

r24755: {projects} final changes for 1.0.5 (projects/android/trunk/Orbot)
by Nathan Freitas 16 May '11

16 May '11

Author: n8fr8 Date: 2011-05-16 03:52:10 +0000 (Mon, 16 May 2011) New Revision: 24755 Modified: projects/android/trunk/Orbot/CHANGELOG Log: final changes for 1.0.5 Modified: projects/android/trunk/Orbot/CHANGELOG =================================================================== --- projects/android/trunk/Orbot/CHANGELOG 2011-05-16 03:51:08 UTC (rev 24754) +++ projects/android/trunk/Orbot/CHANGELOG 2011-05-16 03:52:10 UTC (rev 24755) @@ -1,6 +1,7 @@ NOTE: Specific #s below correspond to Trac tickets logged and maintained at https://trac.torproject.org/projects/tor/ 1.0.5 +- added exit node and "StrictExitNode" preference - fixed tor binary installation issue related to max resource size and compression - updated "start on boot" code to test for proper launch event - updated to Tor 0.2.2.25-alpha binary

1 0

r24754: {} adding tor control java lib to project (projects/android/trunk/Orbot/libs)
by Nathan Freitas 16 May '11

16 May '11

Author: n8fr8 Date: 2011-05-16 03:51:08 +0000 (Mon, 16 May 2011) New Revision: 24754 Added: projects/android/trunk/Orbot/libs/jtorctrl.jar Log: adding tor control java lib to project Added: projects/android/trunk/Orbot/libs/jtorctrl.jar =================================================================== (Binary files differ) Property changes on: projects/android/trunk/Orbot/libs/jtorctrl.jar ___________________________________________________________________ Added: svn:executable + * Added: svn:mime-type + application/octet-stream

1 0

r24753: {projects} added exit node option to preferences (in projects/android/trunk/Orbot/src/org/torproject/android: . service)
by Nathan Freitas 16 May '11

16 May '11

Author: n8fr8 Date: 2011-05-16 03:50:10 +0000 (Mon, 16 May 2011) New Revision: 24753 Modified: projects/android/trunk/Orbot/src/org/torproject/android/SettingsPreferences.java projects/android/trunk/Orbot/src/org/torproject/android/service/TorService.java projects/android/trunk/Orbot/src/org/torproject/android/service/TorTransProxy.java Log: added exit node option to preferences Modified: projects/android/trunk/Orbot/src/org/torproject/android/SettingsPreferences.java =================================================================== --- projects/android/trunk/Orbot/src/org/torproject/android/SettingsPreferences.java 2011-05-16 03:49:54 UTC (rev 24752) +++ projects/android/trunk/Orbot/src/org/torproject/android/SettingsPreferences.java 2011-05-16 03:50:10 UTC (rev 24753) @@ -77,7 +77,7 @@ } - int hiddenGroupIdx = 5; + int hiddenGroupIdx = 6; prefHiddenServices = ((CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(hiddenGroupIdx)).getPreference(0)); prefHiddenServices.setOnPreferenceClickListener(this); Modified: projects/android/trunk/Orbot/src/org/torproject/android/service/TorService.java =================================================================== --- projects/android/trunk/Orbot/src/org/torproject/android/service/TorService.java 2011-05-16 03:49:54 UTC (rev 24752) +++ projects/android/trunk/Orbot/src/org/torproject/android/service/TorService.java 2011-05-16 03:50:10 UTC (rev 24753) @@ -1288,6 +1288,10 @@ boolean enableHiddenServices = prefs.getBoolean("pref_hs_enable", false); + boolean enableStrictExitNodes = prefs.getBoolean("pref_strict_exit_nodes", false); + String exitNodes = prefs.getString("pref_exit_nodes", ""); + + //boolean enableTransparentProxy = prefs.getBoolean(TorConstants.PREF_TRANSPARENT, false); if (currentStatus == STATUS_ON) @@ -1303,6 +1307,14 @@ logException("unable to setup transproxy",e); } } + + if (exitNodes.length() > 0) + { + + mBinder.updateConfiguration("ExitNodes", exitNodes, false); + mBinder.updateConfiguration("StrictExitNodes", enableStrictExitNodes ? "1" : "0", false); + + } if (useBridges) { Modified: projects/android/trunk/Orbot/src/org/torproject/android/service/TorTransProxy.java =================================================================== --- projects/android/trunk/Orbot/src/org/torproject/android/service/TorTransProxy.java 2011-05-16 03:49:54 UTC (rev 24752) +++ projects/android/trunk/Orbot/src/org/torproject/android/service/TorTransProxy.java 2011-05-16 03:50:10 UTC (rev 24753) @@ -335,15 +335,17 @@ script.append(ipTablesPath); script.append(" -I FORWARD"); - script.append(" -s 192.168.43.0/24 -j ACCEPT"); + script.append(" -j ACCEPT"); script.append(" || exit\n"); + /* script.append(ipTablesPath); script.append(" -P FORWARD DROP"); script.append(" || exit\n"); + */ script.append(ipTablesPath); - script.append(" -t nat -I POSTROUTING -s 192.168.43.0/24 -j MASQUERADE"); + script.append(" -t nat -I POSTROUTING -j MASQUERADE"); script.append(" || exit\n"); String[] cmdAdd = {script.toString()}; @@ -372,8 +374,8 @@ purgeIptables(context); - //enableWifiHotspotRules(context); - + enableWifiHotspotRules(context); + int torUid = context.getApplicationInfo().uid; // Set up port redirection @@ -472,6 +474,8 @@ String msg = res.toString(); TorService.logMessage(cmdAdd[0] + ";errCode=" + code + ";resp=" + msg); + + return code; }

1 0

r24752: {projects} updated manifest with version numbers (projects/android/trunk/Orbot)
by Nathan Freitas 16 May '11

16 May '11

Author: n8fr8 Date: 2011-05-16 03:49:54 +0000 (Mon, 16 May 2011) New Revision: 24752 Modified: projects/android/trunk/Orbot/AndroidManifest.xml.debug projects/android/trunk/Orbot/AndroidManifest.xml.production Log: updated manifest with version numbers Modified: projects/android/trunk/Orbot/AndroidManifest.xml.debug =================================================================== --- projects/android/trunk/Orbot/AndroidManifest.xml.debug 2011-05-16 03:49:36 UTC (rev 24751) +++ projects/android/trunk/Orbot/AndroidManifest.xml.debug 2011-05-16 03:49:54 UTC (rev 24752) @@ -1,6 +1,6 @@ <?xml version="1.0" encoding="utf-8"?> <manifest xmlns:android="http://schemas.android.com/apk/res/android" - package="org.torproject.android" android:versionName="1.0.4" android:versionCode="10"> + package="org.torproject.android" android:versionName="0.2.2.25-orbot-alpha-1.0.5.20110503a-dev-debug" android:versionCode="10"> <uses-permission android:name="android.permission.INTERNET" /> Modified: projects/android/trunk/Orbot/AndroidManifest.xml.production =================================================================== --- projects/android/trunk/Orbot/AndroidManifest.xml.production 2011-05-16 03:49:36 UTC (rev 24751) +++ projects/android/trunk/Orbot/AndroidManifest.xml.production 2011-05-16 03:49:54 UTC (rev 24752) @@ -1,6 +1,6 @@ <?xml version="1.0" encoding="utf-8"?> <manifest xmlns:android="http://schemas.android.com/apk/res/android" - package="org.torproject.android" android:versionName="0.2.2.22-orbot-alpha-1.0.5.20110416a-dev" android:versionCode="11"> + package="org.torproject.android" android:versionName="0.2.2.25-orbot-alpha-1.0.5.20110503a" android:versionCode="11"> <uses-permission android:name="android.permission.INTERNET" /> <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED"/>

1 0

r24751: {projects} updated strings files with version number (in projects/android/trunk/Orbot/res: layout values values-ar values-ca values-de values-es values-fa values-mk values-nb values-nl values-pl values-pt values-ru values-sv values-zh xml)
by Nathan Freitas 16 May '11

16 May '11

Author: n8fr8 Date: 2011-05-16 03:49:36 +0000 (Mon, 16 May 2011) New Revision: 24751 Modified: projects/android/trunk/Orbot/res/layout/layout_about.xml projects/android/trunk/Orbot/res/layout/layout_main.xml projects/android/trunk/Orbot/res/values-ar/strings.xml projects/android/trunk/Orbot/res/values-ca/strings.xml projects/android/trunk/Orbot/res/values-de/strings.xml projects/android/trunk/Orbot/res/values-es/strings.xml projects/android/trunk/Orbot/res/values-fa/strings.xml projects/android/trunk/Orbot/res/values-mk/strings.xml projects/android/trunk/Orbot/res/values-nb/strings.xml projects/android/trunk/Orbot/res/values-nl/strings.xml projects/android/trunk/Orbot/res/values-pl/strings.xml projects/android/trunk/Orbot/res/values-pt/strings.xml projects/android/trunk/Orbot/res/values-ru/strings.xml projects/android/trunk/Orbot/res/values-sv/strings.xml projects/android/trunk/Orbot/res/values-zh/strings.xml projects/android/trunk/Orbot/res/values/strings.xml projects/android/trunk/Orbot/res/xml/preferences.xml Log: updated strings files with version number Modified: projects/android/trunk/Orbot/res/layout/layout_about.xml =================================================================== --- projects/android/trunk/Orbot/res/layout/layout_about.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/layout/layout_about.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -80,7 +80,7 @@ android:paddingLeft="15px" android:textStyle="bold" android:textColor="#00ff00" /> - <TextView android:text="Tor v0.2.2.14: https://www.torproject.org" + <TextView android:text="Tor v0.2.2.25: https://www.torproject.org" android:layout_width="fill_parent" android:layout_height="wrap_content" android:autoLink="web" Modified: projects/android/trunk/Orbot/res/layout/layout_main.xml =================================================================== --- projects/android/trunk/Orbot/res/layout/layout_main.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/layout/layout_main.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -15,8 +15,8 @@ <ImageView android:id="@+id/radioModeImage" android:layout_width="wrap_content" android:layout_height="wrap_content" - android:layout_marginTop="0px" - android:layout_marginRight="3px" + android:layout_marginTop="3dip" + android:layout_marginRight="3dip" android:gravity="right" android:layout_toRightOf="@+id/radioModeLabel" android:layout_alignParentRight="true" @@ -34,10 +34,6 @@ android:textColor="#333333" /> </RelativeLayout> - <ScrollView android:id="@+id/mainview" - android:orientation="vertical" - android:layout_width="fill_parent" - android:layout_height="fill_parent"> <TableLayout android:id="@+id/mainLayout" android:layout_gravity="center" android:layout_height="wrap_content" @@ -50,6 +46,7 @@ <TableRow> <ImageView android:id="@+id/imgStatus" + android:layout_marginTop="30dip" android:layout_width="wrap_content" android:layout_height="wrap_content" android:src="@drawable/toroff"/> @@ -60,8 +57,8 @@ android:layout_gravity="center_horizontal" android:gravity="center_horizontal" android:textStyle="bold" - android:width="330px" - android:height="75px" + android:layout_width="fill_parent" + android:layout_height="75px" android:textColor="#ffffff" /> </TableRow> @@ -69,7 +66,6 @@ </TableRow> </TableLayout> - </ScrollView> Modified: projects/android/trunk/Orbot/res/values/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,8 +1,8 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> - + <string name="app_version">1.0.5</string> + <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> <string name="secure_default_web_url">https://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-ar/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-ar/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-ar/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">اوربوت (Orbot)</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-ca/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-ca/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-ca/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-de/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-de/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-de/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-es/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-es/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-es/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-fa/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-fa/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-fa/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">اوربات</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-mk/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-mk/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-mk/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Орбот</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-nb/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-nb/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-nb/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-nl/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-nl/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-nl/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-pl/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-pl/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-pl/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-pt/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-pt/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-pt/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> <string name="secure_default_web_url">https://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-ru/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-ru/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-ru/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-sv/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-sv/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-sv/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/values-zh/strings.xml =================================================================== --- projects/android/trunk/Orbot/res/values-zh/strings.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/values-zh/strings.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -1,7 +1,7 @@ <?xml version="1.0" encoding="utf-8"?> <resources> <string name="app_name">Orbot</string> - <string name="app_version">1.0.5-dev</string> + <string name="app_version">1.0.5</string> <string name="internal_web_url">http://orbot/</string> <string name="default_web_url">http://check.torproject.org</string> Modified: projects/android/trunk/Orbot/res/xml/preferences.xml =================================================================== --- projects/android/trunk/Orbot/res/xml/preferences.xml 2011-05-14 19:14:00 UTC (rev 24750) +++ projects/android/trunk/Orbot/res/xml/preferences.xml 2011-05-16 03:49:36 UTC (rev 24751) @@ -42,7 +42,19 @@ </PreferenceCategory> - +<PreferenceCategory android:title="Exit Nodes"> + +<EditTextPreference android:key="pref_exit_nodes" +android:title="Exit Nodes" +android:summary="Fingerprints, nicknames, country codes and address patterns for the last hop" +android:dialogTitle="Enter Exit Nodes" +/> + +<CheckBoxPreference android:defaultValue="false" +android:title="Strict Exit Nodes" android:key="pref_strict_exit_nodes" +android:summary="Use only these specific exit nodes"/> + +</PreferenceCategory> <PreferenceCategory android:title="Bridges"> <CheckBoxPreference android:defaultValue="false"

1 0

[tor/maint-0.2.2] Increase the length of the buffer in smartlist_string_num_isin().
by nickm＠torproject.org 16 May '11

16 May '11

commit 83fe07d3f26aff703230305c9ff529592af32d4f Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun May 15 22:13:53 2011 -0400 Increase the length of the buffer in smartlist_string_num_isin(). This was harmless, since we only used this for checking for lists of port values, but it's the principle of the thing. Fixes 3175; bugfix on 0.1.0.1-rc --- changes/bug3175 | 7 +++++++ src/common/container.c | 2 +- 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/changes/bug3175 b/changes/bug3175 new file mode 100644 index 0000000..3360fbc --- /dev/null +++ b/changes/bug3175 @@ -0,0 +1,7 @@ + o Minor bugfixes: + - Resolve an untriggerable issue in smartlist_string_num_isin(), + where if the function had ever in the future been used to check + for the presence of a too-large number, it would have given an + incorrect result. (Fortunately, we only used it for 16-bit + values.) Fixes bug 3175; bugfix on Tor 0.1.0.1-rc. + diff --git a/src/common/container.c b/src/common/container.c index ca49cbb..da44b7f 100644 --- a/src/common/container.c +++ b/src/common/container.c @@ -210,7 +210,7 @@ smartlist_string_isin_case(const smartlist_t *sl, const char *element) int smartlist_string_num_isin(const smartlist_t *sl, int num) { - char buf[16]; + char buf[32]; /* long enough for 64-bit int, and then some. */ tor_snprintf(buf,sizeof(buf),"%d", num); return smartlist_string_isin(sl, buf); }

1 0

[tor/master] Merge remote-tracking branch 'origin/maint-0.2.2'
by nickm＠torproject.org 16 May '11

16 May '11

commit 4ac8ff9c9f182b6202200480f8fc4689da659331 Merge: f2c1702 ede102f Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun May 15 20:22:44 2011 -0400 Merge remote-tracking branch 'origin/maint-0.2.2' changes/bug2792_checkdir | 8 ++++++ changes/bug2972 | 5 ++++ doc/tor.1.txt | 5 ++++ src/common/compat.c | 39 +++++++++++++++++++++++++++++++ src/common/compat.h | 2 + src/common/util.c | 58 ++++++++++++++++++++++++++++++++++++++-------- src/common/util.h | 7 ++++- src/or/config.c | 13 ++++++++-- src/or/connection.c | 49 ++++++++++++++++++++++++++++++++++++++ src/or/or.h | 1 + src/test/test_util.c | 30 +++++++++++++++++++++++ 11 files changed, 203 insertions(+), 14 deletions(-)

1 0

[tor/master] Make check_private_dir check for group ownership as appropriate
by nickm＠torproject.org 16 May '11

16 May '11

commit f72e792be5437c9ee11d3f498ed3bb469b46d1bb Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun May 15 13:11:48 2011 -0400 Make check_private_dir check for group ownership as appropriate --- src/common/util.c | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/src/common/util.c b/src/common/util.c index 0e739f2..1bb116b 100644 --- a/src/common/util.c +++ b/src/common/util.c @@ -30,6 +30,7 @@ #else #include <dirent.h> #include <pwd.h> +#include <grp.h> #endif /* math.h needs this on Linux */ @@ -1736,6 +1737,21 @@ check_private_dir(const char *dirname, cpd_check_t check) tor_free(process_ownername); return -1; } + if ((check & CPD_GROUP_OK) && st.st_gid != getgid()) { + struct group *gr; + char *process_groupname = NULL; + gr = getgrgid(getgid()); + process_groupname = gr ? tor_strdup(gr->gr_name) : tor_strdup("<unknown>"); + gr = getgrgid(st.st_gid); + + log_warn(LD_FS, "%s is not owned by this group (%s, %d) but by group " + "%s (%d). Are you running Tor as the wrong user?", + dirname, process_groupname, (int)getgid(), + gr ? gr->gr_name : "<unknown>", (int)st.st_gid); + + tor_free(process_groupname); + return -1; + } if (check & CPD_GROUP_OK) { mask = 0027; } else {

1 0

[tor/master] Merge branch 'bug2972' into maint-0.2.2
by nickm＠torproject.org 16 May '11

16 May '11

commit ede102fd4675af8ac4060767f0348f3c93617e80 Merge: 2b9c5ee f72e792 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sun May 15 20:20:36 2011 -0400 Merge branch 'bug2972' into maint-0.2.2 changes/bug2792_checkdir | 8 ++++++ changes/bug2972 | 5 ++++ doc/tor.1.txt | 5 ++++ src/common/compat.c | 39 +++++++++++++++++++++++++++++++ src/common/compat.h | 2 + src/common/util.c | 58 ++++++++++++++++++++++++++++++++++++++-------- src/common/util.h | 7 ++++- src/or/config.c | 13 ++++++++-- src/or/connection.c | 49 ++++++++++++++++++++++++++++++++++++++ src/or/or.h | 1 + src/test/test_util.c | 30 +++++++++++++++++++++++ 11 files changed, 203 insertions(+), 14 deletions(-)

1 0

[tor/master] Clean up the 2972 implementation a little
by nickm＠torproject.org 16 May '11

16 May '11

commit 4198261291c8edbd5ba1617b7bfe3563e51edbe7 Author: Sebastian Hahn <sebastian(a)torproject.org> Date: Sat Apr 23 02:57:53 2011 +0200 Clean up the 2972 implementation a little --- changes/bug2972 | 5 +++++ doc/tor.1.txt | 2 +- src/or/config.c | 14 ++++++++++---- src/or/connection.c | 4 ++-- src/or/or.h | 2 +- 5 files changed, 19 insertions(+), 8 deletions(-) diff --git a/changes/bug2972 b/changes/bug2972 new file mode 100644 index 0000000..26afcca --- /dev/null +++ b/changes/bug2972 @@ -0,0 +1,5 @@ + o Minor features: + - Allow ControlSockets to be group-writable when the + ControlSocksGroupWritable configuration option is turned on. Patch + by Jérémy Bobbio; implements ticket 2972. + diff --git a/doc/tor.1.txt b/doc/tor.1.txt index d0d0c2f..1815a8d 100644 --- a/doc/tor.1.txt +++ b/doc/tor.1.txt @@ -167,7 +167,7 @@ Other options can be specified either on the command-line (--option Like ControlPort, but listens on a Unix domain socket, rather than a TCP socket. (Unix and Unix-like systems only.) -**UnixSocketsGroupWritable** **0**|**1**:: +**ControlSocketsGroupWritable** **0**|**1**:: If this option is set to 0, don't allow the filesystem group to read and write unix sockets (e.g. ControlSocket). If the option is set to 1, make the control socket readable and writable by the default GID. (Default: 0) diff --git a/src/or/config.c b/src/or/config.c index c81fc9c..614fc48 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -209,7 +209,7 @@ static config_var_t _option_vars[] = { V(ControlPortFileGroupReadable,BOOL, "0"), V(ControlPortWriteToFile, FILENAME, NULL), V(ControlSocket, LINELIST, NULL), - V(UnixSocketsGroupWritable, BOOL, "0"), + V(ControlSocketsGroupWritable, BOOL, "0"), V(CookieAuthentication, BOOL, "0"), V(CookieAuthFileGroupReadable, BOOL, "0"), V(CookieAuthFile, STRING, NULL), @@ -953,9 +953,15 @@ options_act_reversible(or_options_t *old_options, char **msg) } #ifndef HAVE_SYS_UN_H - if (options->ControlSocket || options->UnixSocketsGroupWritable) { - *msg = tor_strdup("Unix domain sockets (ControlSocket) not supported" - " on this OS/with this build."); + if (options->ControlSocket || options->ControlSocketsGroupWritable) { + *msg = tor_strdup("Unix domain sockets (ControlSocket) not supported " + "on this OS/with this build."); + goto rollback; + } +#else + if (options->ControlSocketsGroupWritable && !options->ControlSocket) { + *msg = tor_strdup("Setting ControlSocketGroupWritable without setting" + "a ControlSocket makes no sense."); goto rollback; } #endif diff --git a/src/or/connection.c b/src/or/connection.c index d0898c5..12e00e5 100644 --- a/src/or/connection.c +++ b/src/or/connection.c @@ -966,9 +966,9 @@ connection_create_listener(const struct sockaddr *listensockaddr, tor_socket_strerror(tor_socket_errno(s))); goto err; } - if (get_options()->UnixSocketsGroupWritable) { + if (get_options()->ControlSocketsGroupWritable) { if (chmod(address, 0660) < 0) { - log_warn(LD_FS,"Unable to make %s group-readable.", address); + log_warn(LD_FS,"Unable to make %s group-writable.", address); tor_close_socket(s); goto err; } diff --git a/src/or/or.h b/src/or/or.h index b72693f..b9d8319 100644 --- a/src/or/or.h +++ b/src/or/or.h @@ -2445,7 +2445,7 @@ typedef struct { int ControlPort; /**< Port to listen on for control connections. */ config_line_t *ControlSocket; /**< List of Unix Domain Sockets to listen on * for control connections. */ - int UnixSocketsGroupWritable; /**< Boolean: Are unix sockets g+rw? */ + int ControlSocketsGroupWritable; /**< Boolean: Are control sockets g+rw? */ int DirPort; /**< Port to listen on for directory connections. */ int DNSPort; /**< Port to listen on for DNS requests. */ int AssumeReachable; /**< Whether to publish our descriptor regardless. */

1 0