github / github/codeql

Java: `Location.getFile().getAbsolutePath()` contains Windows drive letter for non-source classes

Aperta
#5,712 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Java question
Lingua principale
CodeQL
Stelle
10.1k
Fork
2.1k
Merge medio
2g 15h
PR unite (30g)
141

Descrizione

### Version
CodeQL CLI version 2.5.0

### Description
For types which are not part of the source, `Location.getFile().getAbsolutePath()` returns a path which contains the Windows drive letter.
For tests which have the path (as part of the URL) in their expected output this therefore results in test failures on Windows.
For example if I run on Windows the test added by #5711 it fails due to the file URL mismatch.

However, it looks like there are no existing tests in this repository with non-source types in their exepcted output, and CodeQL CLI explicitly warns about this:
> Locations outside the test directory do not work well for regression tests.

Though it is still rather weird why the Windows drive letter is included in the first place.

### Example
Code QL query:
```ql
import java

from TypeObject t
select t, t.getLocation().getFile().getAbsolutePath()
```
Under Windows with the database being under `C:\...` the result is: C:/modules/java.base/java/lang/Object.class
Under Linux the result is: `/modules/java.base/java/lang/Object.class`

(Note: For some demo projects the path is `/usr/lib/jvm/java-8-openjdk-amd64/jre/lib/rt.jar/java/lang/Object.class` ([Query Console link](https://lgtm.com/query/61030082203192419/)); I am not sure on what this depends and whether this could cause issues for other users as well)

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.