Re: [edk2-devel] CodeQL Analysis in edk2

["Laszlo Ersek" <lersek@redhat.com>]

sorry, unfinished thought:

On 11/13/23 14:39, Laszlo Ersek wrote:

> - the "sarif emacs" output seems a bit broken, actually, so it's not usable. Consider the following entry from the original JSON file:
> 
>     }, {
>       "ruleId" : "cpp/missing-null-test",
>       "ruleIndex" : 0,
>       "rule" : {
>         "id" : "cpp/missing-null-test",
>         "index" : 0
>       },
>       "message" : {
>         "text" : "Value may be null; it should be checked before dereferencing."
>       },
>       "locations" : [ {
>         "physicalLocation" : {
>           "artifactLocation" : {
>             "uri" : "MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c",
>             "uriBaseId" : "%SRCROOT%",
>             "index" : 0
>           },
>           "region" : {
>             "startLine" : 355,
>             "startColumn" : 48,
>             "endColumn" : 52
>           }
>         }
>       } ],
>       "partialFingerprints" : {
>         "primaryLocationLineHash" : "f374f6e6dfc92010:1",
>         "primaryLocationStartColumnFingerprint" : "43"
>       }
>     }, {
> 
> In the "emacs" output, it appears as:
> 
> --------
> ModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c:355: cpp/missing-null-test Value may be null; it should be checked before dereferencing.
> --------
> 
> Note that the first three characters, "Mde" of "Mde" are lost.

I meant '"Mde" of "ModulePkg"'.

> 
> This issue (first three chars cut) affects all other pathnames in the emacs output too.




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111154): https://edk2.groups.io/g/devel/message/111154
Mute This Topic: https://groups.io/mt/102444916/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-