On 01.12.22 17:25, coverity-bot wrote: > > This is an experimental semi-automated report about issues detected by > Coverity from a scan of next-20221201 as part of the linux-next scan project: > https://scan.coverity.com/projects/linux-next-weekly-scan > > You're getting this email because you were associated with the identified > lines of code (noted below) that were touched by commits: > > Fri Dec 14 17:59:46 2018 +0100 > 55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files") > > Coverity reported the following: > > [...] > > If this is a false positive, please let us know so we can mark it as > such, or teach the Coverity rules to be smarter. If not, please make > sure fixes get into linux-next. :) For patches fixing this, please > include these lines (but double-check the "Fixes" first): > > Reported-by: coverity-bot <keescook+coverity-bot@xxxxxxxxxxxx> Inspired by Sean's reply yesterday a "nit on the bot" from my side, too: Reports like this will sometimes generate a discussion, hence it IMHO would be good if the bot would suggest to place a "Link:" tag to the report itself right after the "Reported-by:" tag. That might be a bit tricky, but with a bit of luck the bot might already know the msgid of the msg it is about to sent, which makes it straight forward to predict the link the report will later have in the lore archives. To explain: These links make things easier for future code archaeologists. That's why Linus[1] considers proper link tags important and why our documentation for some time says to place tags in cases like this, too[2]. [1] for details, see: https://lore.kernel.org/all/CAHk-=wjMmSZzMJ3Xnskdg4+GGz=5p5p+GSYyFBTh0f-DgvdBWg@xxxxxxxxxxxxxx/ https://lore.kernel.org/all/CAHk-=wgs38ZrfPvy=nOwVkVzjpM3VFU1zobP37Fwd_h9iAD5JQ@xxxxxxxxxxxxxx/ https://lore.kernel.org/all/CAHk-=wjxzafG-=J8oT30s7upn4RhBs6TX-uVFZ5rME+L5_DoJA@xxxxxxxxxxxxxx/ [2] see Documentation/process/submitting-patches.rst (http://docs.kernel.org/process/submitting-patches.html) and Documentation/process/5.Posting.rst (https://docs.kernel.org/process/5.Posting.html) > Addresses-Coverity-ID: 1527765 ("Error handling issues") > Fixes: 55d2375e58a6 ("KVM: nVMX: Move nested code to dedicated files") > [...] Ciao, Thorsten
