Skip to content
Browse files

fixed: ineffective ALSA HDMI device whitelist check

The check added in c7015ec was ineffective, as noted by FernetMenta
(some " == 0" checks went missing after the previous working version of
the patch).

Fix it.
  • Loading branch information...
1 parent 3027c4c commit 20a8c7ccf32fd9699dee4a107db49b76763d036c @anssih anssih committed with Mike Jul 22, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 xbmc/cores/AudioEngine/Sinks/AESinkALSA.cpp
View
2 xbmc/cores/AudioEngine/Sinks/AESinkALSA.cpp
@@ -933,7 +933,7 @@ void CAESinkALSA::EnumerateDevice(AEDeviceInfoList &list, const std::string &dev
/* only trust badHDMI (= unconnected or non-existent port) on Intel
* and NVIDIA where it has been confirmed to work, show the empty
* port on other systems */
- if (info.m_displayName.compare(0, 9, "HDA Intel") || info.m_displayName.compare(0, 10, "HDA NVidia"))
+ if (info.m_displayName.compare(0, 9, "HDA Intel") == 0 || info.m_displayName.compare(0, 10, "HDA NVidia") == 0)
{
/* unconnected HDMI port */
CLog::Log(LOGDEBUG, "CAESinkALSA - Skipping HDMI device \"%s\" as it has no ELD data", device.c_str());

0 comments on commit 20a8c7c

Please sign in to comment.
Something went wrong with that request. Please try again.