Ubuntu
kdetv package

kdetv: bug in dcop function "navigate" from teletext interface

Bug #211747 reported by BongoMan
6
Affects Status Importance Assigned to Milestone
kdetv (Ubuntu)
Confirmed
Low
Unassigned

Bug Description

Binary package hint: kdetv

Hello,

I'm running kubuntu 7.10 with kdetv 0.8.9-1 (you can find the sources here: https://launchpad.net/ubuntu/+source/kdetv/0.8.9-1)

The dcop function "TelexIface navigate(int page)" in kdetv doesn't accept the argument "100", which is actually the most important page to navigate to in teletext. I found the error in kdetv/plugins/misc/telex/telex.cpp:

void Plugin::navigate( int page )
    {
        if ( page > 100 )
            navigate( Page( page ) );
    }

The if-condition should be: if ( page > 99 )

I attached a patch which changes the 100 to 99.

ciao,
Pascal Pollet

Tags: patch
Revision history for this message
BongoMan (pascal-bongosoft) wrote :
Revision history for this message
Daniel T Chen (crimsun) wrote :

Is this symptom still reproducible in 8.10 or 9.04?

Changed in kdetv:
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
BongoMan (pascal-bongosoft) wrote :

yes, it is in 8.10. Im am now running Intrepid and had to patch and compile the sources again for this to work. For 9.04, I don't know...

Jonathan Thomas (echidnaman)
Changed in kdetv (Ubuntu):
status: Incomplete → New
Revision history for this message
BongoMan (pascal-bongosoft) wrote :

Hey, sometimes I don't understand how things work in Launchpad. This bug is now older than one year, it has changed it's status twice and is now marked as "new". What is new about that? It is a clear bug and I provided an exact description and the solution. It is one line to change in the sources. Why does someone package this for new distribution releases (already 4 times since initial report) and does not correct the bug before? It would take ~30 seconds. Sorry, I don't get that, what's reporting bugs good for?

Revision history for this message
Thomas Hotz (thotz-deactivatedaccount) wrote :

Setting to confirmed.

Changed in kdetv (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.