793523 - Bookmarks Search doesn't work on big endian after 64-bit folder ID change

Status: RESOLVED FIXED

(Toolkit :: Places, defect)

Description

[Cameron Kaiser [:spectre]]

Attachment: Use %lld when nsPrintfCString'ing the parent ID

"TenFourFox: I Like Big Endian and I Cannot Lie"

Upstreamed from http://code.google.com/p/tenfourfox/issues/detail?id=177

After bug 702639, bookmarks search stopped working on big endian, including TenFourFox and derivatives on PowerPC. NSPR logging showed that the parent IDs were all appearing as zero in the generated SQL. This is probably an endian issue since the same code worked when they were 32-bit. Simply changing the %d to %lld in the attached patch should suffice for this instance, although there are probably others.

Comment 1

[Marco Bonardo [:mak]]

Comment on attachment 663847 [details] [diff] [review]
Use %lld when nsPrintfCString'ing the parent ID

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

Thank you, this was indeed wrong :(
If you need help pushing this just let me know.

Comment 2

[Cameron Kaiser [:spectre]]

I don't have checkin or try permissions -- does this need a try run? Otherwise, yes, please. Thank you!

Comment 3

[Marco Bonardo [:mak]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7026b17114f9

Comment 4

[Marco Bonardo [:mak]]

Comment on attachment 663847 [details] [diff] [review]
Use %lld when nsPrintfCString'ing the parent ID

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 702639
User impact if declined: bookmarks not working properly on big-endian platforms
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): Trivial change, no risk
String or UUID changes made by this patch: none

note: while this doesn't meet usual approval regarding priority, it's so trivial and risk free I think may be worth to get it earlier.

Comment 5

[Marco Bonardo [:mak]]

thanks
https://hg.mozilla.org/releases/mozilla-aurora/rev/d4d1d9fedb30

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/7026b17114f9