In main func, strdup lo.source may fail. So check whether strdup
lo.source return NULL before using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools/virtiofsd/passthrough_ll.c | 4
1 file changed, 4 insertions(+)
diff --git a/tools/virtiofsd/passthrough_ll.c b/tools
In main func, func lo_map_reserve is called without NULL check.
If reallocing new_elems fails in func lo_map_grow, the func
lo_map_reserve may return NULL. We should check whether
lo_map_reserve returns NULL before using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools
In fuse_bufvec_advance func, calling fuse_bufvec_current func
may return NULL, so we should check whether buf is NULL before
using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools/virtiofsd/buffer.c | 4
1 file changed, 4 insertions(+)
diff --git a/tools/virtiofsd
Hi,
We find some potential NULL pointer bugs on tools/virtiofsd.
Three patches are made to fix them
Haotian Li (3):
tools/virtiofsd/buffer.c: check whether buf is NULL in
fuse_bufvec_advance func
virtiofsd: check whether lo_map_reserve returns NULL in main func
virtiofsd: check
Thanks for your suggestion. New patches will be resent.
> On Tue, Nov 10, 2020 at 02:32:17PM +0800, Haotian Li wrote:
>> Haotian Li (2):
>> tools/virtiofsd/buffer.c: check whether buf is NULL in
>> fuse_bufvec_advance func
>> virtiofsd/passthrough_ll.c: c
In main func, func lo_map_reserve is called without NULL check.
If reallocing new_elems fails in func lo_map_grow, the func
lo_map_reserve may return NULL. We should check whether
lo_map_reserve returns NULL before using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools
In fuse_bufvec_advance func, calling fuse_bufvec_current func
may return NULL, so we should check whether buf is NULL before
using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools/virtiofsd/buffer.c | 4
1 file changed, 4 insertions(+)
diff --git a/tools/virtiofsd
Haotian Li (2):
tools/virtiofsd/buffer.c: check whether buf is NULL in
fuse_bufvec_advance func
virtiofsd/passthrough_ll.c: check whether lo_map_reserve returns NULL
in main func
tools/virtiofsd/buffer.c | 4
tools/virtiofsd/passthrough_ll.c | 10 +-
2 files
Hi,
Thanks for your suggestion. We will fix the coding style
problems and resend new patches.
> Patchew URL:
> https://patchew.org/QEMU/eeb2fd1d-a53d-eae6-4727-7f1a6b20a...@huawei.com/
>
>
>
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more informatio
In main func, func lo_map_reserve is called without NULL check.
If reallocing new_elems fails in func lo_map_grow, the func
lo_map_reserve may return NULL. We should check whether
lo_map_reserve returns NULL before using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools
In fuse_bufvec_advance func, calling fuse_bufvec_current func
may return NULL, so we should check whether buf is NULL before
using it.
Signed-off-by: Haotian Li
Signed-off-by: Zhiqiang Liu
---
tools/virtiofsd/buffer.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/virtiofsd
Hi,
We find some potential NULL pointer bugs on tools/virtiofsd.
Two patches are made to fix them.
Haotian Li (2):
tools/virtiofsd/buffer.c: check whether buf is NULL in
fuse_bufvec_advance func
virtiofsd/passthrough_ll.c: check whether lo_map_reserve returns NULL
in main func
12 matches
Mail list logo