Skip to content

Fix: added some null pointer dereference checks - #1931

Open
wavyfuncollaps wants to merge 4 commits into
microsoft:developfrom
wavyfuncollaps:fix/null-pointer-dereference-checks
Open

wavyfuncollaps wants to merge 4 commits into
microsoft:developfrom
wavyfuncollaps:fix/null-pointer-dereference-checks

Conversation

@wavyfuncollaps

Copy link
Copy Markdown

These changes harden the GradleBuildServerBuildSupport class against four null-dereference scenarios:

  • Skip project dependencies that do not map to an Eclipse workspace project.
  • Reject non-file output URIs when conversion produces no filesystem path.
  • Skip JVM build targets with missing or invalid decoded metadata.
  • Skip Maven dependencies with missing or invalid decoded metadata.
    Each case writes a meaningful log message containing relevant identifiers before safely continuing or returning.

@wenytang-ms

Copy link
Copy Markdown
Contributor

wavyfuncollaps thank you for your contribution, can you add some test cases for this pr?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants