[vdr] polarization character case in channels.conf

Stefan Huelswitt s.huelswitt at gmx.de
Thu Nov 8 20:43:44 CET 2012


Hi,

I can confirm this problem, at least in 1.7.27 (which I'm using at the 
moment). I use the attached patch to resolve it.

Am 08.11.2012 10:19, schrieb Klaus Schmidinger:
>>> Well, there is a toupper() in cDvbTransponderParameters::Parse(), so
>>> I would expect that all characters can be given in either upper- or
>>> lowercase.

Right, but the toupper() is only in the comparision, but it passes the 
original (lowercase) value. And later during tuning the comparision fails.

Regards.

-- 
Stefan Huelswitt
s.huelswitt at gmx.de  | http://www.muempf.de/
-------------- next part --------------
diff -uN -x .dependencies -x vdr.pc -x Make.config -x '*.o' -x '*.rej' vdr-1.7.27-orig/dvbdevice.c vdr-current-1.7.27/dvbdevice.c
--- vdr-1.7.27-orig/dvbdevice.c	2012-03-25 12:41:45.000000000 +0200
+++ vdr-current-1.7.27/dvbdevice.c	2012-07-21 19:12:57.542055052 +0200
@@ -769,7 +771,7 @@
            frequency -= Setup.LnbFrequHi;
            tone = SEC_TONE_ON;
            }
-        int volt = (dtp.Polarization() == 'V' || dtp.Polarization() == 'R') ? SEC_VOLTAGE_13 : SEC_VOLTAGE_18;
+        int volt = (dtp.Polarization() == 'V' || dtp.Polarization() == 'R' || dtp.Polarization() == 'v' || dtp.Polarization() == 'r') ? SEC_VOLTAGE_13 : SEC_VOLTAGE_18;
         if (GetBondedMaster() != this) {
            tone = SEC_TONE_OFF;
            volt = SEC_VOLTAGE_13;


More information about the vdr mailing list