Skip to content

L
lxc
Unverified Commit 09ef5d95 by Stéphane Graber Committed by GitHub
Options

Merge pull request #2505 from brauner/2018-07-31/bugfixes

coverity: #438136
parents a3cf04da aeba3f80
Showing with 3 additions and 2 deletions
src/lxc/tools/lxc_destroy.c
...@@ -211,6 +211,7 @@ static bool do_destroy_with_snapshots(struct lxc_container *c) ...@@ -211,6 +211,7 @@ static bool do_destroy_with_snapshots(struct lxc_container *c)
char *lxcname = NULL; char *lxcname = NULL;
int fd; int fd;
int ret; int ret;
ssize_t bytes;
/* Destroy clones. */ /* Destroy clones. */
ret = snprintf(path, MAXPATHLEN, "%s/%s/lxc_snapshots", c->config_path, c->name); ret = snprintf(path, MAXPATHLEN, "%s/%s/lxc_snapshots", c->config_path, c->name);
...@@ -233,9 +234,9 @@ static bool do_destroy_with_snapshots(struct lxc_container *c) ...@@ -233,9 +234,9 @@ static bool do_destroy_with_snapshots(struct lxc_container *c)
return false; return false;
} }
ret = lxc_read_nointr(fd, buf, fbuf.st_size); bytes = lxc_read_nointr(fd, buf, fbuf.st_size);
close(fd); close(fd);
if (ret < 0) { if (bytes != (ssize_t)fbuf.st_size) {
ERROR("Could not read %s", path); ERROR("Could not read %s", path);
free(buf); free(buf);
return false; return false;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment