Re: [PATCH] hda empty modelname comparison

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

 



On Tue, 2007-03-06 at 15:59 +0100, Joachim Deguara wrote:
This patch prevents a comparison between a possibly empty modelname
string in the hda code.

-Joachim

Unfortunately, the way the original patch was written, the only options are generic or segfault.  Your patch just removes the segfault, sending all cards to the generic parser.

I've tested a slightly different approach, based on your empty test, just reverse logic.  Test it to confirm that it works.  It works on my system with a Conexant reference board and a Sigmatel STAC9227 (both on the same system).



Signed off by Tobin Davis <tdavis@xxxxxxxxxxxx>
diff -r bf193701a812 pci/hda/hda_codec.c
--- a/pci/hda/hda_codec.c	Mon Feb 26 16:07:42 2007 +0100
+++ b/pci/hda/hda_codec.c	Wed Mar 07 12:02:37 2007 -0800
@@ -573,7 +573,7 @@ int snd_hda_codec_new(struct hda_bus *bu
 							 0);
 	}
 
-	if (strcmp(codec->bus->modelname, "generic"))
+	if (!(codec->bus->modelname && strcmp(codec->bus->modelname, "generic")))
 		codec->preset = find_codec_preset(codec);
 	if (! *bus->card->mixername)
 		snd_hda_get_codec_name(codec, bus->card->mixername,
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux