Closed Bug 695965 Opened 8 years ago Closed 8 years ago

valgrind: Conditional jump or move depends on uninitialised value(s) in nsUrlClassifierDBService::CheckClean

Categories

(Core :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: jaas, Assigned: jaas)

Details

Attachments

(1 file, 1 obsolete file)

1.87 KB, patch
bent.mozilla
: review+
Details | Diff | Splinter Review
Attached patch fix v1.0 (obsolete) — Splinter Review
==29775== Conditional jump or move depends on uninitialised value(s)
==29775==    at 0x1083BF985: nsUrlClassifierDBService::CheckClean(nsACString_internal const&, bool*) (nsUrlClassifierDBService.cpp:1439)
==29775==    by 0x1083C9649: nsUrlClassifierDBService::LookupURI(nsIURI*, nsIUrlClassifierCallback*, bool, bool*) (nsUrlClassifierDBService.cpp:4218)
==29775==    by 0x1083C9CBE: nsUrlClassifierDBService::Classify(nsIURI*, nsIURIClassifierCallback*, bool*) (nsUrlClassifierDBService.cpp:4165)
==29775==    by 0x10721C50B: nsChannelClassifier::Start(nsIChannel*) (nsChannelClassifier.cpp:123)
==29775==    by 0x107333E43: nsHttpChannel::AsyncOpen(nsIStreamListener*, nsISupports*) (nsHttpChannel.cpp:3763)
==29775==    by 0x1082EA5C5: nsURILoader::OpenURI(nsIChannel*, bool, nsIInterfaceRequestor*) (nsURILoader.cpp:815)
==29775==    by 0x1082B924E: nsDocShell::DoChannelLoad(nsIChannel*, nsIURILoader*, bool) (nsDocShell.cpp:9076)
==29775==    by 0x1082BE67F: nsDocShell::DoURILoad(nsIURI*, nsIURI*, bool, nsISupports*, char const*, nsIInputStream*, nsIInputStream*, bool, nsIDocShell**, nsIRequest**, bool, bool, bool) (nsDocShell.cpp:8913)
==29775==    by 0x1082D4EA9: nsDocShell::InternalLoad(nsIURI*, nsIURI*, nsISupports*, unsigned int, unsigned short const*, char const*, nsIInputStream*, nsIInputStream*, unsigned int, nsISHEntry*, bool, nsIDocShell**, nsIRequest**) (nsDocShell.cpp:8616)
==29775==    by 0x1082BF919: nsDocShell::LoadURI(nsIURI*, nsIDocShellLoadInfo*, unsigned int, bool) (nsDocShell.cpp:1462)
==29775==    by 0x1082C3E0A: nsDocShell::LoadURI(unsigned short const*, unsigned int, nsIURI*, nsIInputStream*, nsIInputStream*) (nsDocShell.cpp:3790)
==29775==    by 0x10895B624: NS_InvokeByIndex_P (xptcinvoke_x86_64_unix.cpp:195)
Attachment #568286 - Flags: review?(bent.mozilla)
I don't know who is in charge of this code and it probably doesn't matter much so I picked an arbitrary reviewer. Feel free to reassign review if you want.
Comment on attachment 568286 [details] [diff] [review]
fix v1.0

I think you should fix Probe too... "found" is a return parameter and it's a violation of XPCOM's contract for a function to return NS_OK without setting its return parameters. This is just waiting to bite someone else.
Attached patch fix v1.1Splinter Review
Attachment #568286 - Attachment is obsolete: true
Attachment #568286 - Flags: review?(bent.mozilla)
Attachment #568293 - Flags: review?(bent.mozilla)
Attachment #568293 - Flags: review?(bent.mozilla) → review+
Keywords: checkin-needed
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/103807cd0432
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.