gustavonihei commented on pull request #695: URL: https://github.com/apache/incubator-nuttx-apps/pull/695#issuecomment-839761694
> > > > By the way, instead of undervaluing the reviewer's methods, please explain clearly on why an issue is not valid. If I am still wrong on the other raised issues, I'd like to learn from them. > > > > > > > > > if the code is bug-free, the value doesn't matter. > > > they should not be used by anyone. > > > thus those NULLify should not be necessary. > > > > > > The value wouldn't matter if it were stored in a local variable, but it is not the case. Since the lifetime of this dangling pointer is extended, the value does matter. > > And all it takes for a software to change from "Bug-free" to "Vulnerable" is one single commit. > > And if this single commit naively operates on this dangling pointer, the system will crash. > > why doesn't a local variable matter? > with your logic, i think the value left on stack might be unintentionally used by a buggy software. > Well, I was referring to local variables that are about to go out-of-scope, that's why I mentioned the extended lifetime for the other cases that the value does matter. If a local variable goes out-of-scope (i.e. it's lifetime is over), it cannot be referenced anymore, so its pointer value is harmless. But I won't bother if you still want to nullify it. > > > i pushed a counter-proposal to the defensive style. [219356c](https://github.com/apache/incubator-nuttx-apps/commit/219356ce7d1b8a26140c98d303d493c6f04b19e9) > > > > > > Good, this commit may be a proof that this code is not **currently** buggy. But that's not the issue I raised. > > Now, here it is a simple unit test for making this potential bug evident: > > https://godbolt.org/z/f4dzW8M44 > > the unit test is just broken. > it doesn't seem to support your argument. Oh, really? Guess why it is broken? ``` Program stderr free(): double free detected in tcache 2 ``` I rest my case. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
