Message ID | 20240722081736.20439-8-xni@redhat.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | mdadm: fix coverity issues | expand |
Context | Check | Description |
---|---|---|
mdraidci/vmtest-md-6_11-PR | success | PR summary |
mdraidci/vmtest-md-6_11-VM_Test-0 | success | Logs for build-kernel |
diff --git a/mdopen.c b/mdopen.c index eaa59b59..c9fda131 100644 --- a/mdopen.c +++ b/mdopen.c @@ -406,7 +406,11 @@ int create_mddev(char *dev, char *name, int autof, int trustworthy, perror("chown"); if (chmod(devname, ci->mode)) perror("chmod"); - stat(devname, &stb); + if (stat(devname, &stb) < 0) { + pr_err("failed to stat %s\n", + devname); + return -1; + } add_dev(devname, &stb, 0, NULL); } if (use_mdp == 1)
It needs to check return values when functions return value. Signed-off-by: Xiao Ni <xni@redhat.com> --- mdopen.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)