Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

wasi: remove unecessary null check #42819

Merged
merged 1 commit into from
Apr 24, 2022
Merged

wasi: remove unecessary null check #42819

merged 1 commit into from
Apr 24, 2022

Commits on Apr 21, 2022

  1. wasi: remove unecessary null check 

    As reported by coverity the null check is
done after earlier code has already derefenced
options.envp. Either we should null check earlier
uses or this check is not necessary. Since options.envp
is created with new which should throw an
exception intead of returning null, it should
be safe to assume the null check is not required.

Signed-off-by: Michael Dawson <[email protected]>
    @mhdawson
    mhdawson committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    27f2b3b View commit details
    Browse the repository at this point in the history