|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] libxc: tidy up loop construct in write_compressed
commit fcaacad1a472f3023a7fd9cea4132c3f6533c913
Author: Matthew Daley <mattjd@xxxxxxxxx>
AuthorDate: Wed Oct 30 20:51:41 2013 +1300
Commit: Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Thu Oct 31 21:23:49 2013 +0000
libxc: tidy up loop construct in write_compressed
...otherwise the return 0 at the bottom is dead code.
Coverity-ID: 1055189
Signed-off-by: Matthew Daley <mattjd@xxxxxxxxx>
Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
tools/libxc/xc_domain_save.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/libxc/xc_domain_save.c b/tools/libxc/xc_domain_save.c
index 673d1e9..42c4752 100644
--- a/tools/libxc/xc_domain_save.c
+++ b/tools/libxc/xc_domain_save.c
@@ -233,7 +233,7 @@ static int write_compressed(xc_interface *xch, comp_ctx
*compress_ctx,
int marker = XC_SAVE_ID_COMPRESSED_DATA;
unsigned long compbuf_len = 0;
- do
+ for(;;)
{
/* check for available space (atleast 8k) */
if ((ob->pos + header + XC_PAGE_SIZE * 2) > ob->size)
@@ -250,7 +250,7 @@ static int write_compressed(xc_interface *xch, comp_ctx
*compress_ctx,
ob->size - ob->pos - header,
&compbuf_len);
if (!rc)
- return 0;
+ break;
if (outbuf_hardwrite(xch, ob, fd, &marker, sizeof(marker)) < 0)
{
@@ -270,7 +270,7 @@ static int write_compressed(xc_interface *xch, comp_ctx
*compress_ctx,
ERROR("Error when writing compressed chunk");
return -1;
}
- } while (rc != 0);
+ }
return 0;
}
--
generated by git-patchbot for /home/xen/git/xen.git#master
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |