Re: [PATCH v1 1/2] Fixes: Coverity CID 1558827

2024-08-08 Thread Peter Maydell
On Tue, 6 Aug 2024 at 14:50, Chalapathi V wrote: The Subject email for a patch should say what it does in terms of code changes, not just list the bug number or coverity issue number, and it should start with the prefix showing what part of the codebase it is changing. You can look through other

Re: [PATCH v1 1/2] Fixes: Coverity CID 1558827

2024-08-06 Thread Philippe Mathieu-Daudé
Hi Chalapathi, Please prefix subject with "hw/ssi/pnv". On 6/8/24 15:48, Chalapathi V wrote: In this commit the following coverity scan defect has been fixed. CID 1558827:(OVERRUN) Overrunning array "s->seq_op" of 8 bytes at byte offset 16 using index "get_seq_index(s) + 1" (which evalua

[PATCH v1 1/2] Fixes: Coverity CID 1558827

2024-08-06 Thread Chalapathi V
In this commit the following coverity scan defect has been fixed. CID 1558827:(OVERRUN) Overrunning array "s->seq_op" of 8 bytes at byte offset 16 using index "get_seq_index(s) + 1" (which evaluates to 16). Signed-off-by: Chalapathi V --- hw/ssi/pnv_spi.c | 6 +++--- 1 file changed, 3 inse