Project

General

Profile

Bug #15430

Feature #14568: Additional Software Packages

Feature #14594: Code & integration into Tails: Additional Software Packages GUI

live-additional-software.conf is given wrong permissions when it's updated by tails-additional-software

Added by bertagaz about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
03/20/2018
Due date:
% Done:

100%

QA Check:
Pass
Feature Branch:
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Additional Software Packages

Description

With an ISO build with 8dd945efef3e4f47f11742862cf34b098ae42cc2

  • Boot from USB with network and admin password with no configured persistent storage
  • Install a package with APT
  • Confirm you want to create the persistent storage
  • Enter passphrases and confirm
  • Wait for t-p-s to finish and close it

/media/tails-persistence-setup/TailsData/live-additional-software.conf is 0600

live-persist rightfully detects it's not 0644 on next boot and disable ASP.

Associated revisions

Revision 6b8f5d66 (diff)
Added by alant about 1 year ago

ASP: pythonlib with fixed rights

Will-fix: #15430

Revision 3d588395 (diff)
Added by bertagaz about 1 year ago

Test suite: remove workaround for #15430.

Fixed in ASP.

Refs: #14596, #15430

Revision 4eb23799 (diff)
Added by bertagaz 8 months ago

Test suite: remove workaround for #15430.

Fixed in ASP.

Refs: #14596, #15430

History

#1 Updated by bertagaz about 1 year ago

  • Parent task set to #14595

#2 Updated by intrigeri about 1 year ago

/media/tails-persistence-setup/TailsData/live-additional-software.conf is 0600

Strange, config/chroot_local-patches/tps-14594-asp-gui.diff has code to make it 0644. I'll take a look today or tomorrow!

#3 Updated by bertagaz about 1 year ago

intrigeri wrote:

/media/tails-persistence-setup/TailsData/live-additional-software.conf is 0600

Strange, config/chroot_local-patches/tps-14594-asp-gui.diff has code to make it 0644. I'll take a look today or tomorrow!

Yesh, that's why I've set this ticket as new. I'm not quite sure of what's is happening, and I'm surprised given what I vaguely understand from the t-p-s patch, so I wonder if I'm messing up. But I have this failure with the automated test and I can reproduce it by hand.

#4 Updated by intrigeri about 1 year ago

  • Subject changed from T-p-s does not create live-additional-software.conf with the right permissions to live-additional-software.conf is given wrong permissions when it's updated by tails-additional-software
  • Status changed from New to Confirmed
  • Assignee changed from intrigeri to alant

I've reproduced this but I believe that t-p-s is not at fault: if I create a persistent storage with t-p-s, the file has correct permissions (0644). So I'm inclined to blame additionalsoftware/config.py's _write_config in our Python library. Feel free to reassign to me if you see that the permissions are wrong after t-p-s exits but before your code touches the file.

#5 Updated by intrigeri about 1 year ago

  • Parent task changed from #14595 to #14594

#6 Updated by alant about 1 year ago

intrigeri wrote:

So I'm inclined to blame additionalsoftware/config.py's _write_config in our Python library.

You're right. This should be fixed by commit 6b8f5d6.

#7 Updated by alant about 1 year ago

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA

#8 Updated by bertagaz about 1 year ago

According to the test suite, this is fixed. I was able to remove the workaround there was about this bug and have the test suite still passing.

#9 Updated by alant about 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

Also available in: Atom PDF