]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
ALSA: hda: tas2781: Fix wrong reference of tasdevice_priv
authorTakashi Iwai <tiwai@suse.de>
Wed, 20 Aug 2025 05:19:01 +0000 (07:19 +0200)
committerTakashi Iwai <tiwai@suse.de>
Wed, 20 Aug 2025 05:23:23 +0000 (07:23 +0200)
During the conversion to unify the calibration data management, the
reference to tasdevice_priv was wrongly set to h->hda_priv instead of
h->priv.  This resulted in memory corruption and crashes eventually.
Unfortunately it's a void pointer, hence the compiler couldn't know
that it's wrong.

Fixes: 4fe238513407 ("ALSA: hda/tas2781: Move and unified the calibrated-data getting function for SPI and I2C into the tas2781_hda lib")
Link: https://bugzilla.suse.com/show_bug.cgi?id=1248270
Cc: <stable@vger.kernel.org>
Link: https://patch.msgid.link/20250820051902.4523-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/hda/codecs/side-codecs/tas2781_hda_i2c.c

index b91fff3fde97fbbe804d5015707f0454bef3e9e7..e34b17f0c9b9e2eec40be808059dcea94f91d745 100644 (file)
@@ -305,7 +305,7 @@ static int tas2563_save_calibration(struct tas2781_hda *h)
        efi_char16_t efi_name[TAS2563_CAL_VAR_NAME_MAX];
        unsigned long max_size = TAS2563_CAL_DATA_SIZE;
        unsigned char var8[TAS2563_CAL_VAR_NAME_MAX];
-       struct tasdevice_priv *p = h->hda_priv;
+       struct tasdevice_priv *p = h->priv;
        struct calidata *cd = &p->cali_data;
        struct cali_reg *r = &cd->cali_reg_array;
        unsigned int offset = 0;