[PATCH] covoar: Fix NULL check of wrong variable (CID #1399602)

Gedare Bloom gedare at rtems.org
Wed Apr 7 17:57:08 UTC 2021


ok

On Wed, Apr 7, 2021 at 11:47 AM Alex White <alex.white at oarcorp.com> wrote:
>
> CID 1399602: Dereference null return value in
> Explanations::writeNotFound().
>
> In Explanations::writeNotFound() there were two NULL checks of the
> `fileName` variable where only one is needed. The second check has been
> changed to a NULL check of `notFoundFile` to match the original intent.
>
> Closes #4377
> ---
>  tester/covoar/Explanations.cc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tester/covoar/Explanations.cc b/tester/covoar/Explanations.cc
> index 7b0f95a..12bd809 100644
> --- a/tester/covoar/Explanations.cc
> +++ b/tester/covoar/Explanations.cc
> @@ -140,10 +140,10 @@ done:
>        return;
>
>      notFoundFile = fopen( fileName, "w" );
> -    if (!fileName) {
> +    if (notFoundFile == nullptr) {
>        std::ostringstream what;
>        what << "Unable to open " << fileName
> -           << "out of sync at the explanation";
> +           << " out of sync at the explanation";
>        throw rld::error( what, "Explanations::writeNotFound" );
>      }
>
> --
> 2.27.0
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list