Fixing Coverity Defects Part Deux#1130
Conversation
There was a problem hiding this comment.
Mono-style hits LibGit2Sharp for 42 damage points.
|
@Therzok update made, can you please re-review? 🙏 |
|
Looks good to me otherwise 👍 |
|
@Therzok 🙇 thank you sir. @nulltoken you're up: what's your opinion about |
|
This needs a rebase. |
… where the issues can occur.
…or code to the callback invoker.
Done. Not painful - sorry to disappoint 😜 |
There was a problem hiding this comment.
I'm not sure about. This will brutally flow back to libgit2.
Is there even a case when currentChange could actually be null?
There was a problem hiding this comment.
Is there even a case when currentChange could actually be null?
@nulltoken Coverity sure seems to think there is. Perhaps in some kind of repo corruption case? Not sure really. Null checking doesn't seem like a bad thing. Very little overhead and it would be correct to route and error back to the base lib if the value were null.
Another round of fixes for Coverity discovered defects.
Please closely review the NRE fixes, as there are likely better ways to resolve them that I have no considered.