mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-22 04:25:18 +00:00
lxc_fuse: Use automatic file closing
There are two functions (lxcProcHostRead() and lxcProcReadMeminfo()) that could benefit from automatic file closing. Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
parent
d68a8b2109
commit
8b36a2574f
@ -126,7 +126,7 @@ lxcProcHostRead(char *path,
|
||||
size_t size,
|
||||
off_t offset)
|
||||
{
|
||||
int fd;
|
||||
VIR_AUTOCLOSE fd = -1;
|
||||
int res;
|
||||
|
||||
fd = open(path, O_RDONLY);
|
||||
@ -136,7 +136,6 @@ lxcProcHostRead(char *path,
|
||||
if ((res = pread(fd, buf, size, offset)) < 0)
|
||||
res = -errno;
|
||||
|
||||
VIR_FORCE_CLOSE(fd);
|
||||
return res;
|
||||
}
|
||||
|
||||
@ -148,7 +147,7 @@ lxcProcReadMeminfo(char *hostpath,
|
||||
off_t offset)
|
||||
{
|
||||
int res;
|
||||
FILE *fp = NULL;
|
||||
g_autoptr(FILE) fp = NULL;
|
||||
g_autofree char *line = NULL;
|
||||
size_t n;
|
||||
struct virLXCMeminfo meminfo;
|
||||
@ -251,7 +250,6 @@ lxcProcReadMeminfo(char *hostpath,
|
||||
memcpy(buf, virBufferCurrentContent(new_meminfo), res);
|
||||
|
||||
cleanup:
|
||||
VIR_FORCE_FCLOSE(fp);
|
||||
return res;
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user