1511647 - Investigate removing NULL checks on aArgv

Status: RESOLVED WORKSFORME

(Core :: IPC, task, P3)

Description

[Darkspirit]

See bug 1477037 for whole context.

(Nathan Froyd [:froydnj] from bug 1477037 comment 15)
> Can you file a followup bug in Core::IPC about removing these checks?  I don't see any reason that we should be passing nullptr in the arg list...unless we're processing the nullptr at the end of argv, which indicates a logic bug someplace else.

Comment 1

[Jed Davis [:jld] ⟨⏰|UTC-7⟩ ⟦he/him⟧]

See also bug 1509813, which is the same problem of mystery nulls on regular builds.  This also means that the guess in bug 1477037 comment #10, that it's related to --disable-crashreporter, is probably not it.

Bug 1477037 comment #5 has a reliable repro, if I understand correctly, so if that reproduces under rr it should be relatively simple to reverse-debug with watchpoints to find out where the null is coming from.

Failing that, printing out the argument list and seeing where the null is could give some clues.

Comment 2

[Jed Davis [:jld] ⟨⏰|UTC-7⟩ ⟦he/him⟧]

ni? me to see if there's a reliable repro

Comment 3

[:gerard-majax]

This is code we removed in bug 1723505

Comment 4

[:gerard-majax]

So, this code is dead, and building tree as of bug 1477037 is not working anymore, so reproducing that is kinda hard. All IPC level CLI parsing should have moved to CheckArg() that is immune to nullptr so there should be no reason to crash anymore ; and disabling crash reporter does not show any dangling nullptr within aArgv.