On 28/07/2021 18:27, Joel Sherrill wrote:
The commit message should explain that.
I will try to improve the commit message.
I don't see a CID for this one in the reports but someone should check
that. I don't like their web interface and might have missed something.
I noticed this while re
On Wed, Jul 28, 2021 at 11:23 AM Gedare Bloom wrote:
>
> On Wed, Jul 28, 2021 at 9:13 AM Joel Sherrill wrote:
> >
> > Is this a Coverity issue? Does it have a CID?
> >
> > I remembered TOCTOU as an acronym analysis reports used
> > but had to look up the meaning. I don't know what to do so someon
On Wed, Jul 28, 2021 at 9:13 AM Joel Sherrill wrote:
>
> Is this a Coverity issue? Does it have a CID?
>
> I remembered TOCTOU as an acronym analysis reports used
> but had to look up the meaning. I don't know what to do so someone
> else would not have to look it up beyond explaining it in the lo
Is this a Coverity issue? Does it have a CID?
I remembered TOCTOU as an acronym analysis reports used
but had to look up the meaning. I don't know what to do so someone
else would not have to look it up beyond explaining it in the long comment.
It is missing what was checked and used that flagged
Fix format, add Doxygen comments, and reduce includes.
---
cpukit/libcsupport/src/getchark.c | 21 +
1 file changed, 13 insertions(+), 8 deletions(-)
diff --git a/cpukit/libcsupport/src/getchark.c
b/cpukit/libcsupport/src/getchark.c
index 5a9afbda7a..cfe9c022d6 100644
--- a/c