Ticket #74 (closed Defect: fixed)

Opened 3 months ago

Last modified 3 months ago

whitelist the few (one?) env variables that autopackage needs

Reported by: Jan-Nik Assigned to: Jan-Nik
Priority: Medium Milestone: 1.2.5
Component: main Version: 1.2.4
Severity: Critical Keywords:
Cc:

Description

Mailing list discussion

I had this problem some months ago but now I'm running into it again.

Attachments

reset_env.diff (383 bytes) - added by Jan-Nik on 04/26/08 13:17:10.
Here's a patch. I'm sorry, didn't have the time to test it. Maybe someone can do that for me?

Change History

04/26/08 12:55:47 changed by Jan-Nik

  • severity changed from Major to Critical.
  • milestone changed from 1.2.6 to 1.2.5.

Another mailing list discussion

04/26/08 13:17:10 changed by Jan-Nik

  • attachment reset_env.diff added.

Here's a patch. I'm sorry, didn't have the time to test it. Maybe someone can do that for me?

04/28/08 14:18:38 changed by isak

  • status changed from new to closed.
  • resolution set to fixed.

This fixes it. Committed in r2441. Thanks again for your work!

I'm closing this as fixed, but we may get bitten by this in the future in case the default sudo config changes:

From sudo(8) manpage:

-E The -E (preserve environment) option will override the env_reset option in sudoers(5)). It is only available when either the matching command has the SETENV tag or the setenv option is set in sudoers(5).