• Aiswarya Cyriac's avatar
    ALSA: virtio: Fix "Coverity: virtsnd_kctl_tlv_op(): Uninitialized variables" warning. · ba00e413
    Aiswarya Cyriac authored
    
    
    This commit fixes the following warning when building virtio_snd driver.
    
    "
    *** CID 1583619:  Uninitialized variables  (UNINIT)
    sound/virtio/virtio_kctl.c:294 in virtsnd_kctl_tlv_op()
    288
    289     		break;
    290     	}
    291
    292     	kfree(tlv);
    293
    vvv     CID 1583619:  Uninitialized variables  (UNINIT)
    vvv     Using uninitialized value "rc".
    294     	return rc;
    295     }
    296
    297     /**
    298      * virtsnd_kctl_get_enum_items() - Query items for the ENUMERATED element type.
    299      * @snd: VirtIO sound device.
    "
    
    This warning is caused by the absence of the "default" branch in the
    switch-block, and is a false positive because the kernel calls
    virtsnd_kctl_tlv_op() only with values for op_flag processed in
    this block.
    
    Also, this commit unifies the cleanup path for all possible control
    paths in the callback function.
    Signed-off-by: default avatarAnton Yakovlev <anton.yakovlev@opensynergy.com>
    Signed-off-by: Aiswarya Cyriac <aiswarya.cyri...
    ba00e413
virtio_kctl.c 13 KB