New TAP to manage the PVR using HTML/CSS/JavaScript.

[DeltaMikeCharlie]

A new version has been posted.

• I have finished including all of the INF data into the files listing.
• I have updated the recordings list to show the name of the event (from the INF).
• I have also updated jkIT's demo to show the channel name. (The sort by channel does not work at the moment)
• I have experimented with converting the channel name to UTF8 using one of FireBird's functions. This should show on the recordings page in the "Channel Name" column. Hopefully, the characters look better now. I only did this field because I could use a hex editor to modify an INF for testing. I will update the channels listing tomorrow, but I can not test it.

[Isarwolf]

I new version has been posted.

With this version I have also reboots.

[DeltaMikeCharlie]

[quote="Isarwolf"]With this version I have also reboots.[/quote]

Can you check the log file and find out where the reboot occurs?

[Isarwolf]

[quote="DeltaMikeCharlie"]Can you check the log file and find out where the reboot occurs?[/quote]

I have made a log with two crash.

[DeltaMikeCharlie]

That is very odd, it rebooted in different places each time. Did you change anything between the 2 tests? The first test seems to be with the demo and the second test with the default page. Both show 39 recordings. Did you rename any files?

I have uploaded a new version that produces a lot more logging to help identify where the problem may be occurring. Please rerun your tests and let me know where they fail. Remember, the log is erased each time TMA1 is restarted.

I do not recommend that anyone use this version unless they are debugging this rebooting issue. The logging overhead could slow things down a lot.

[Sirius]

With the A018(new) version I have also a reboot by Records list.
And my channel list was empty - (but this is not so important at the moment).

Here are my logs of A018 and A018(new)

The new version A019 from post #165 I will test later...

[DeltaMikeCharlie]

@Sirius: Yours is even worse, it reboots before any files are read. Even before the number of files to read is known!!

This is very odd, I do not get these symptoms.

This all started when I added the INF data. My working theory is that you have some characters that are causing a problem, or perhaps your INF file has subtle differences that cause the reboots. If we can narrow down the problem to some specific recordings, please post the INFs so that I can take a closer look. This may be true for Isarwolf, but Sirius has no data yet and it still reboots.

I'll check any new logs posted tomorrow morning (my time) and see if anything helpful shows up.

Thanks to all for helping with this one.

[FireBird]

I've just tried A019 and while the channel list is empty, I don't get a reboot on the other pages. If there's something I can test, then I need to do quickly because starting tomorrow, I'll be away for the next two weeks.

[Sirius]

With A019 the log ends also with this line:

Code: Alles auswählen

Processing API-Files-Get request '/api?function=files&action=get!dir=/DataFiles&'.

Freeze and after 2 seconds the reboot ... before any file is read.

[Isarwolf]

"All good things go by threes".

[DeltaMikeCharlie]

@Isarwolf

The reboot is caused by a simple buffer overflow. The allocation that I make for the JSON returned is too small. But it's not that simple.

Something is very wrong with the INF file and/or the STRUCT that I use to read it.

I am seeing negative numbers for bookmark locations. Also, there should only be a maximum of 177 bookmarks, whereas your PVR is reporting 17771 or 15056, etc.

Can you please post a copy of the following INF?

"Bloch Die Geisel - 2013-03-09 20-12.rec.inf"
"Das Leuchten der Stille - 2013-08-20 02-17.rec.inf"

Also perhaps also a few more random ones of your choosing.

If you want to send via email instead, send me a PM and I will send you my email address.

@Firebird - Are you aware of INFs from different models having different formats?

Towards the end of the log, I am seeing dates like "32009/109/101 49:49:00" which are clearly wrong. For now, I am attributing that to a side-effect of the buffer overflow because at the beginning of the log, those dates seem OK.

[DeltaMikeCharlie]

[quote="FireBird"]I've just tried A019 and while the channel list is empty[/quote]

When you type in the channels command manually, do you get a listing?

http://<PVR IP>:8000/api?function=channels&action=get!

[DeltaMikeCharlie]

I have uploaded version A-020 that I hope will stop the reboots for now.

I do a simple bounds check on the number of bookmarks that the INF thinks it contains, I only read the bookmarks section if the number is reported as between 0 and 177. The underlying problem still exists, I have just tried to prevent the reboot.

Ultra-detailed logging is still enabled.

[FireBird]

[quote="DeltaMikeCharlie"]Are you aware of INFs from different models having different formats?[/quote]
There are minor differences in the transponder structs of DVBs, -t and -c models but they are at the end of the interessting part (just before the bookmarks) but the function HDD_DecodeRECHeader() should take care about this.

[quote="DeltaMikeCharlie"]When you type in the channels command manually, do you get a listing?

http://<PVR IP>:8000/api?function=channels&action=get![/quote]
IE downloads a file called api.json. But I've just found out that the missing list is only valid for IE. Firefox displays the channels as expected.

[DeltaMikeCharlie]

With A019 the log ends also with this line:

Code: Alles auswählen

Processing API-Files-Get request '/api?function=files&action=get!dir=/DataFiles&'.

Freeze and after 2 seconds the reboot ... before any file is read.

I'm working on a debugging version for this issue too. For now, if you have access to the telnet console, can you try again and look for messages about keywords and directories:

Function = [files], Action = [get], Module = [default], Session = [].
Processing API-Files-Get request '/api?function=files&action=get!dir=/DataFiles&'.
theKeyword = [dir], theValue = [/DataFiles].
fullDir = [/DataFiles]
tempDir = [/DataFiles]

[DeltaMikeCharlie]

[quote="FireBird"]There are minor differences in the transponder structs of DVBs, -t and -c models but they are at the end of the interessting part (just before the bookmarks) but the function HDD_DecodeRECHeader() should take care about this.[/quote]

Would this difference also change the offset of the snapshot images? If so, this could impact INFplus.

[quote="FireBird"]IE downloads a file called api.json. But I've just found out that the missing list is only valid for IE. Firefox displays the channels as expected.[/quote]

I don't recall a file called "api.json" for TMA1. Curious.

[FireBird]

I'm not 100% sure but I think that the previews start at a fixed offset.

[Sirius]

Bad news, the A020 crashes on my toppy when I looked for the Records list.

The Telnet window (putty) shows the 'Alive' message and f.e. Timers output, but when I pressed the Records button my toppy freezes and putty closed immediately (??), so that I can't see the things you asked for.

The log is about 750K big (because of the channel infos).
In the attachment is a small log. This was created when the browser window of TMA was already opened and after restarting I pressed only the Records button.
.

[Isarwolf]

I have three info-files in the Zip-file.
Die Stille I have just deleted yesterday evening.
I have seen that this info files are bigger than normal, normally the have 8,0 KiB, this one have 129,5 KiB. Exactly the same for all.

[DeltaMikeCharlie]

I have uploaded A-021.

• This version has extra-super-improved-mega-logging(TM) for Sirius.
• It also uses FireBird's HDD_DecodeRECHeader to read the INF file and hopefully stop the reboots. It seems to work OK for me, but then so did the previous versions.