[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 5/7] util/qemu-option: Document the get_opt_value() function
From: |
Laurent Vivier |
Subject: |
[PULL 5/7] util/qemu-option: Document the get_opt_value() function |
Date: |
Tue, 7 Jul 2020 12:52:53 +0200 |
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Coverity noticed commit 950c4e6c94 introduced a dereference before
null check in get_opt_value (CID1391003):
In get_opt_value: All paths that lead to this null pointer
comparison already dereference the pointer earlier (CWE-476)
We fixed this in commit 6e3ad3f0e31, but relaxed the check in commit
0c2f6e7ee99 because "No callers of get_opt_value() pass in a NULL
for the 'value' parameter".
Since this function is publicly exposed, it risks new users to do
the same error again. Avoid that documenting the 'value' argument
must not be NULL.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20200629070858.19850-1-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
include/qemu/option.h | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/include/qemu/option.h b/include/qemu/option.h
index eb4097889d52..ac50d257746a 100644
--- a/include/qemu/option.h
+++ b/include/qemu/option.h
@@ -28,6 +28,19 @@
#include "qemu/queue.h"
+/**
+ * get_opt_value
+ * @p: a pointer to the option name, delimited by commas
+ * @value: a non-NULL pointer that will received the delimited options
+ *
+ * The @value char pointer will be allocated and filled with
+ * the delimited options.
+ *
+ * Returns the position of the comma delimiter/zero byte after the
+ * option name in @p.
+ * The memory pointer in @value must be released with a call to g_free()
+ * when no longer required.
+ */
const char *get_opt_value(const char *p, char **value);
void parse_option_size(const char *name, const char *value,
--
2.26.2
- [PULL 0/7] Trivial branch for 5.1 patches, Laurent Vivier, 2020/07/07
- [PULL 1/7] fix the prototype of muls64/mulu64, Laurent Vivier, 2020/07/07
- [PULL 2/7] trivial: Respect alphabetical order of .o files in Makefile.objs, Laurent Vivier, 2020/07/07
- [PULL 3/7] .mailmap: Update Alexander Graf email address, Laurent Vivier, 2020/07/07
- [PULL 4/7] MAINTAINERS: Update Radoslaw Biernacki email address, Laurent Vivier, 2020/07/07
- [PULL 5/7] util/qemu-option: Document the get_opt_value() function,
Laurent Vivier <=
- [PULL 6/7] intel_iommu: "aw-bits" error message still refers to "x-aw-bits", Laurent Vivier, 2020/07/07
- [PULL 7/7] net/tap-solaris.c: Include qemu-common.h for TFR macro, Laurent Vivier, 2020/07/07
- Re: [PULL 0/7] Trivial branch for 5.1 patches, Peter Maydell, 2020/07/09