Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6.x.x] Fix an NPE in exist:time pragma #5081

Merged

Conversation

adamretter
Copy link
Contributor

Avoid a small NPE that can be triggered by #5077

@adamretter adamretter added the bug issue confirmed as bug label Oct 9, 2023
@adamretter adamretter added this to the eXist-6.2.1 milestone Oct 9, 2023
@sonarcloud
Copy link

sonarcloud bot commented Oct 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@line-o
Copy link
Member

line-o commented Oct 9, 2023

This PR adds google find bugs to the dependencies of the range index module. Is this relevant to fixing this NPE?

@adamretter
Copy link
Contributor Author

Is this relevant to fixing this NPE?

It's relevant to making the code compile, so that the NPE can be fixed.

@line-o
Copy link
Member

line-o commented Oct 9, 2023

The tests are failing due to a non-existent CVE file from evolved binary mirror. I was under the impression we do not use that mirror anymore.

@dizzzz dizzzz merged commit 4a94b68 into eXist-db:develop-6.x.x Oct 16, 2023
6 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue confirmed as bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants