1159090 - Only append library path for updater if it is a unique value

Status: RESOLVED FIXED

(Toolkit :: Application Update, defect)

Description

[Brian R. Bondy [:bbondy]]

Bug 1158866 appends to the library path but it should check if the path already exists first.  If Firefox is updated multiple times in sequence without being closed, it can lead to having a long library path.

Comment 1

[Brian R. Bondy [:bbondy]]

Attachment: Patch v1.

Comment 2

[Brian R. Bondy [:bbondy]]

Comment on attachment 8598397 [details] [diff] [review]
Patch v1.

Review of attachment 8598397 [details] [diff] [review]:
-----------------------------------------------------------------

Will resubmit

Comment 3

[Brian R. Bondy [:bbondy]]

Comment on attachment 8598397 [details] [diff] [review]
Patch v1.

Sorry this was fine, I had seen that some builds were failing on oak but it's just from a mozconfig check python script unrelated to this.

Comment 4

[Mark Smith [:mcs]]

I am not sure how likely it is to cause a problem, but simply using strstr() could lead to some false matches.  For example, if LD_LIBRARY_PATH contains /home/user/firefox37 and pathToAppend is /home/user/firefox, then pathToAppend will not be added.  Unless installDirPath.get() always returns a / terminated string?

Comment 5

[Brian R. Bondy [:bbondy]]

That's true but I'm not particularly concerned since these env vars are per process. If this was PATH I'd be more concerned.  Feel free to file a follow up though if you disagree. I'll land this one as is.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6e0c202e4469

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6e0c202e4469