diff options
| author | Sheetal Pamecha <sheetal.pamecha08@gmail.com> | 2018-12-10 18:02:19 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-12-14 04:38:23 +0000 |
| commit | 2421453bf38469a8c0861f205bdee37e771aa39f (patch) | |
| tree | a7b8d763662366add916cc64bb6affa45b964679 /cli | |
| parent | 64d800940e67b75d51f536816ee92bae59d6f850 (diff) | |
cli: variable-length array declaration clang fix
Problem: Declared variable-length array can have zero size
Added a goto statement to address this issue.
Updates: bz#1622665
Change-Id: Ibf80643490367a5f7e50f66f87e4296380be45de
Signed-off-by: Sheetal Pamecha <sheetal.pamecha08@gmail.com>
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/src/cli-cmd-volume.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c index 8df7c2377bc..11dd0735322 100644 --- a/cli/src/cli-cmd-volume.c +++ b/cli/src/cli-cmd-volume.c @@ -852,10 +852,12 @@ out: #if (USE_EVENTS) if (ret == 0 && strcmp(words[2], "help") != 0) { ret1 = dict_get_int32(options, "count", &num_options); - if (ret1) + if (ret1) { num_options = 0; - else + goto end; + } else { num_options = num_options / 2; + } char *free_list_key[num_options]; char *free_list_val[num_options]; @@ -896,6 +898,7 @@ out: } #endif +end: CLI_STACK_DESTROY(frame); return ret; |