465487 - Fix perms on a couple of executable JS components

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Phil Ringnalda (:philor)]

Attachment: mc: nsContentPrefService.js and nsHandlerService.js

No doubt this is a duplicate of half a dozen bugs, since people seem to love filing "OMG LULZ ur perms iz bogus!!1!" bugs much more than they love patching them, but too bad for them. I need a place to fix just these two core/toolkit bits of bug 458467, without hearing about someone's list of other things in some other product.

Comment 1

[(not currently active) Ted Mielczarek]

Comment on attachment 348728 [details] [diff] [review]
mc: nsContentPrefService.js and nsHandlerService.js

Also, you can have blanket r=me to just change permissions where necessary.

Comment 2

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/mozilla-central/rev/675a3a1a22cc

Comment 3

[Henry N.]

(In reply to comment #2)
> http://hg.mozilla.org/mozilla-central/rev/675a3a1a22cc

think, this change has nothing to do with the Bug 465487 here.

Files still are executable:
-rwxr-xr-x 1 root root 53725 Dec 11 11:03 ./components/nsHandlerService.js
-rwxr-xr-x 1 root root 30465 Dec 11 11:03 ./components/nsContentPrefService.js

BuildID:
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.9.1b3pre) Gecko/20081211 Shredder/3.0b2pre

Comment 4

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 348728 [details] [diff] [review]
mc: nsContentPrefService.js and nsHandlerService.js

a191=beltzner

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/releases/mozilla-1.9.1/rev/779729211760