cnescatlab / cnescatlab/i-CodeCNES

False positive of the SH.FLOW.CheckReturn rule

Open
#149 0 comments 0 reactions 0 assignees View on GitHub
false positive shell
Dominant language
Fortran
Stars
63
Forks
18
PR merge metrics
No merged PRs in 30d

Description

### Expected behavior
A blank line between a command and the return check should not raise a violation on the command line

### Actual behavior
A blank line causes a correct check to raise a violation

### Steps to reproduce behavior
cd $HOME

if [ $? -eq 0 ]; then
echo "echec du cd"
exit 1
else
ls
if [ $? -eq 0 ]; then
echo "echec du ls"
exit 1
fi
fi

### Detection version
3.1.0-dev

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.