Opened 12 months ago

Closed 7 months ago

#19953 closed defect (fixed)

DataDirectoryGroupReadable does not take effect when relay enabled

Reported by: redfish Owned by: dgoulet
Priority: Medium Milestone: Tor: 0.3.0.x-final
Component: Core Tor Version: Tor: 0.2.8.6
Severity: Normal Keywords:
Cc: redfish@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Add 'DataDirectoryGroupReadable 1'
Permissions on /var/lib/tor are as expected: 0750

Now, uncomment 'ORPort 9001':
Permissions on /var/lib/tor briefly switch to 0750 but then back to 0700,
despite DataDirectoryGroupReadable being still set to 1.

The patch for Ticket #17562 that adds DataDirectoryGroupReadable option did not change all calls to check_private_dir:
https://trac.torproject.org/projects/tor/attachment/ticket/17562/0002-Introduce-DataDirectoryGroupReadable-boolean.patch

Patch that fixes the issue is attached to this ticket. But, ideally, should factor the check for data dir into a function, but I figured a dev can do that instead of me dabbling with the code.

Child Tickets

Attachments (1)

datadir-group-readable.patch (744 bytes) - added by redfish 12 months ago.
Patch that fixes the issue.

Download all attachments as: .zip

Change History (8)

Changed 12 months ago by redfish

Patch that fixes the issue.

comment:1 Changed 12 months ago by redfish

Cc: redfish@… added

comment:2 Changed 12 months ago by nickm

Milestone: Tor: 0.2.8.x-final
Status: newneeds_review

comment:3 Changed 11 months ago by nickm

Status: needs_reviewneeds_revision

This looks like it should be correct. I'll need a changes file and a unified diff in order to merge it.

comment:4 Changed 9 months ago by nickm

Milestone: Tor: 0.2.8.x-finalTor: 0.3.0.x-final

comment:5 Changed 7 months ago by dgoulet

Owner: set to dgoulet
Status: needs_revisionaccepted

comment:6 Changed 7 months ago by dgoulet

Status: acceptedmerge_ready

I've taken the patch as is and added changes file. The patch does fixes the issue.

See branch bug19953_030_01.

comment:7 Changed 7 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

merged!

Note: See TracTickets for help on using tickets.