+ sound-oss-tridentc-fix-incorrect-test-in-trident_ac97_set.patch added to -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch titled
     sound/oss/trident.c: fix incorrect test in trident_ac97_set()
has been added to the -mm tree.  Its filename is
     sound-oss-tridentc-fix-incorrect-test-in-trident_ac97_set.patch

*** Remember to use Documentation/SubmitChecklist when testing your code ***

See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this

------------------------------------------------------
Subject: sound/oss/trident.c: fix incorrect test in trident_ac97_set()
From: Roel Kluin <12o3l@xxxxxxxxxx>

If count reaches zero, the loop ends, but the postfix decrement still
subtracts: testing for 'count == 0' will not work.

Signed-off-by: Roel Kluin <12o3l@xxxxxxxxxx>
Reviewed-by: Ray Lee <ray-lk@xxxxxxxxxxxxx>
Acked-by: Muli Ben-Yehuda <muli@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 sound/oss/trident.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff -puN sound/oss/trident.c~sound-oss-tridentc-fix-incorrect-test-in-trident_ac97_set sound/oss/trident.c
--- a/sound/oss/trident.c~sound-oss-tridentc-fix-incorrect-test-in-trident_ac97_set
+++ a/sound/oss/trident.c
@@ -2935,7 +2935,7 @@ trident_ac97_set(struct ac97_codec *code
 	do {
 		if ((inw(TRID_REG(card, address)) & busy) == 0)
 			break;
-	} while (count--);
+	} while (--count);
 
 	data |= (mask | (reg & AC97_REG_ADDR));
 
@@ -2996,7 +2996,7 @@ trident_ac97_get(struct ac97_codec *code
 		data = inl(TRID_REG(card, address));
 		if ((data & busy) == 0)
 			break;
-	} while (count--);
+	} while (--count);
 	spin_unlock_irqrestore(&card->lock, flags);
 
 	if (count == 0) {
_

Patches currently in -mm which might be from 12o3l@xxxxxxxxxx are

unlock-when-ssp-tries-to-close-an-invalid-port.patch
git-dvb.patch
git-ia64.patch
git-mips.patch
lmc_ioctl-dont-return-with-locks-held-fix.patch
git-netdev-all.patch
git-ocfs2.patch
cris-gpio-undo-locks-before-returning.patch
mips-undo-locking-on-error-path-returns.patch
mips-undo-locking-on-error-path-returns-checkpatch-fixes.patch
cris-build-fixes-update-eth_v10c-ethernet-driver-fix.patch
sound-oss-tridentc-fix-incorrect-test-in-trident_ac97_set.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Newbies FAQ]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux