diff mbox

[v23,10/32] check NULL input for qemu_opts_del

Message ID 1395396763-26081-11-git-send-email-cyliu@suse.com
State New
Headers show

Commit Message

Chunyan Liu March 21, 2014, 10:12 a.m. UTC 
To simplify later using of qemu_opts_del, accept NULL input.

Signed-off-by: Chunyan Liu <cyliu@suse.com>
---
 util/qemu-option.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Eric Blake March 25, 2014, 9:41 p.m. UTC | #1 
On 03/21/2014 04:12 AM, Chunyan Liu wrote:
> To simplify later using of qemu_opts_del, accept NULL input.
> 
> Signed-off-by: Chunyan Liu <cyliu@suse.com>
> ---
>  util/qemu-option.c | 4 ++++
>  1 file changed, 4 insertions(+)

Reviewed-by: Eric Blake <eblake@redhat.com>

> 
> diff --git a/util/qemu-option.c b/util/qemu-option.c
> index 6c304b2..e9802f3 100644
> --- a/util/qemu-option.c
> +++ b/util/qemu-option.c
> @@ -990,6 +990,10 @@ void qemu_opts_del(QemuOpts *opts)
>  {
>      QemuOpt *opt;
>  
> +    if (opts == NULL) {
> +        return;
> +    }
> +
>      for (;;) {
>          opt = QTAILQ_FIRST(&opts->head);
>          if (opt == NULL)
>
diff mbox

Patch

diff --git a/util/qemu-option.c b/util/qemu-option.c
index 6c304b2..e9802f3 100644
--- a/util/qemu-option.c
+++ b/util/qemu-option.c
@@ -990,6 +990,10 @@  void qemu_opts_del(QemuOpts *opts)
 {
     QemuOpt *opt;
 
+    if (opts == NULL) {
+        return;
+    }
+
     for (;;) {
         opt = QTAILQ_FIRST(&opts->head);
         if (opt == NULL)