Merge lp:~ted/ubuntu-app-launch/tweaks-and-cleanups into lp:ubuntu-app-launch/15.10

Proposed by Ted Gould
Status: Merged
Approved by: Charles Kerr
Approved revision: 203
Merged at revision: 204
Proposed branch: lp:~ted/ubuntu-app-launch/tweaks-and-cleanups
Merge into: lp:ubuntu-app-launch/15.10
Diff against target: 26 lines (+5/-1)
2 files modified
libubuntu-app-launch/ubuntu-app-launch.c (+1/-1)
socket-demangler.c (+4/-0)
To merge this branch: bzr merge lp:~ted/ubuntu-app-launch/tweaks-and-cleanups
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Charles Kerr (community) Approve
Review via email: mp+261522@code.launchpad.net

Commit message

Make demangler path more secure and unset internal environment variables

Description of the change

Thought there was a bug in UAL and fixed a couple things on the way. Turned out not to be an issue in UAL but thought I should push these little fixes out anyway.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
203. By Ted Gould

Switch to g_unsetenv()

Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'libubuntu-app-launch/ubuntu-app-launch.c'
--- libubuntu-app-launch/ubuntu-app-launch.c 2015-06-04 20:22:27 +0000
+++ libubuntu-app-launch/ubuntu-app-launch.c 2015-08-06 21:57:34 +0000
@@ -2515,7 +2515,7 @@
2515 /* The exec value */2515 /* The exec value */
2516 gchar * envstr = NULL;2516 gchar * envstr = NULL;
2517 if (demangler) {2517 if (demangler) {
2518 envstr = g_strdup_printf("APP_EXEC=" DEMANGLER_PATH " %s", execline);2518 envstr = g_strdup_printf("APP_EXEC=%s %s", DEMANGLER_PATH, execline);
2519 } else {2519 } else {
2520 envstr = g_strdup_printf("APP_EXEC=%s", execline);2520 envstr = g_strdup_printf("APP_EXEC=%s", execline);
2521 }2521 }
25222522
=== modified file 'socket-demangler.c'
--- socket-demangler.c 2015-05-21 20:09:44 +0000
+++ socket-demangler.c 2015-08-06 21:57:34 +0000
@@ -120,5 +120,9 @@
120120
121 g_free(mirsocketbuf);121 g_free(mirsocketbuf);
122122
123 /* Don't let people guess about these */
124 g_unsetenv("UBUNTU_APP_LAUNCH_DEMANGLE_NAME");
125 g_unsetenv("UBUNTU_APP_LAUNCH_DEMANGLE_PATH");
126
123 return execvp(argv[1], argv + 1);127 return execvp(argv[1], argv + 1);
124}128}

Subscribers

People subscribed via source and target branches