netbeans icon indicating copy to clipboard operation
netbeans copied to clipboard

prevent NPE, check return value before use

Open homberghp opened this issue 6 months ago • 0 comments

Check returned array value before using, avoiding NPE.

While working on an unrelated subject, I got an error notification that mentions an NPE in the modified file

Simply add a check of the return value before dereferencing it in the following for-each statement. The result is that no elements will be added to the list in the for-each loop, which looks okay to me in that case.

https://github.com/apache/netbeans/blob/68e420dcd08d0e1a3bb7fc1687dd0722881e9422/ide/web.webkit.debugging/src/org/netbeans/modules/web/webkit/debugging/api/Runtime.java#L75-L80


^Add meaningful description above

Click to collapse/expand PR instructions

By opening a pull request you confirm that, unless explicitly stated otherwise, the changes -

  • are all your own work, and you have the right to contribute them.
  • are contributed solely under the terms and conditions of the Apache License 2.0 (see section 5 of the license for more information).

Please make sure (eg. git log) that all commits have a valid name and email address for you in the Author field.

If you're a first time contributor, see the Contributing guidelines for more information.

If you're a committer, please label the PR before pressing "Create pull request" so that the right test jobs can run.

PR approval and merge checklist:

  1. [ ] Was this PR correctly labeled, did the right tests run? When did they run?
  2. [ ] Is this PR squashed?
  3. [ ] Are author name / email address correct? Are co-authors correctly listed? Do the commit messages need updates?
  4. [ ] Does the PR title and description still fit after the Nth iteration? Is the description sufficient to appear in the release notes?

If this PR targets the delivery branch: don't merge. (full wiki article)

homberghp avatar Jun 16 '25 18:06 homberghp