• Arnd Bergmann's avatar
    ALSA: ctxfi: avoid casting function pointers · aabdedf4
    Arnd Bergmann authored
    This driver creates an abstraction for different components by casting function
    pointers to slightly incompatible types for each one to get the correct
    argument even when the caller does not know those types. This is a
    bit unreliable and not allowed in combination with control flow integrity
    (KCFI):
    
    sound/pci/ctxfi/ctatc.c:115:25: error: cast from 'int (*)(struct hw *, struct src_mgr **)' to 'create_t' (aka 'int (*)(struct hw *, void **)') converts to incompatible function type [-Werror,-Wcast-function-type-strict]
      115 |         [SRC]           = { .create     = (create_t)src_mgr_create,
          |                                           ^~~~~~~~~~~~~~~~~~~~~~~~
    sound/pci/ctxfi/ctatc.c:116:20: error: cast from 'int (*)(struct src_mgr *)' to 'destroy_t' (aka 'int (*)(void *)') converts to incompatible function type [-Werror,-Wcast-function-type-strict]
      116 |                             .destroy    = (destroy_t)src_mgr_destroy    },
          |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~
    sound/pci/ctxfi/ctatc.c:117:27: error: cast from 'int (*)(struct hw *, struct srcimp_mgr **)' to 'create_t' (aka 'int (*)(struct hw *, void **)') converts to incompatible function type [-Werror,-Wcast-function-type-strict]
      117 |         [SRCIMP]        = { .create     = (create_t)srcimp_mgr_create,
          |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~
    sound/pci/ctxfi/ctatc.c:118:20: error: cast from 'int (*)(struct srcimp_mgr *)' to 'destroy_t' (aka 'int (*)(void *)') converts to incompatible function type [-Werror,-Wcast-function-type-strict]
      118 |                             .destroy    = (destroy_t)srcimp_mgr_destroy },
          |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    Change these to always pass void pointers and move the abstraction one level
    down.
    
    Fixes: 8cc72361 ("ALSA: SB X-Fi driver merge")
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Link: https://lore.kernel.org/r/20240213101303.460008-1-arnd@kernel.orgSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
    aabdedf4
ctamixer.h 2.66 KB