Incorrect logic / crash in commands.cc

[atuchin-m]

The problem: args_.erase(it); is used instead of it = args_.erase(it);: https://github.com/brave/omaha/blob/1.3.361.113/omaha/base/commands.cc#L565

• erase() is called, there force it is possible invalidated.
• the next comparing of it with args_.end() is undefined behavior (a possible crash)
• it likely fails to erase the next element (how it was designed)

[mherrmann]

As additional context, @atuchin-m clarified that the code was found by @bsclifton via grepping and that he's "not sure whether there’s any real impact on end users".