Jump to content


Photo

[CutListEditor] Crash when called again


  • Please log in to reply
39 replies to this topic

Re: [CutListEditor] Crash when called again #21 WanWizard

  • PLi® Core member
  • 70,204 posts

+1,798
Excellent

Posted 25 October 2021 - 09:46

I've asked again, Erik isn't online much due to health reasons.


Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)

Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.

Many answers to your question can be found in our new and improved wiki.


Re: [CutListEditor] Crash when called again #22 Qu@rk

  • Senior Member
  • 192 posts

+2
Neutral

Posted 25 October 2021 - 10:18

Thanks. I am sorry to hear that, I hope Erik is well soon. That's more important than this problem.

 

Please let me know if you or Erik needs support. But my crashlog (first post) should show everything needed, I hope. 



Re: [CutListEditor] Crash when called again #23 Erik Slagter

  • PLi® Core member
  • 46,969 posts

+541
Excellent

Posted 25 October 2021 - 19:12

I have done quite some work on this source file in the last year, but not in the section the error comes from. It might be related in some way though.

 

I still have the plan to revert all my changes to the file, because the code I added triggers a very ugly bug in glibc, especially on MIPS (less on ARM), but the code is needed for Broadcom transcoding integration. So I will revert all the change and create a filepush class especially for transcoding. That sounds logical and easy as well, but it's quite complex to integrate it into enigma as at various points enigma expects a "filepush" class and not a "transcoding_filepush_class_invented_by_me".

 

It would help if someone could reproduce the issue and actually print the values of both reported variables.


* Wavefrontier T90 with 28E/23E/19E/13E via SCR switches 2 x 2 x 6 user bands
I don't read PM -> if you have something to ask or to report, do it in the forum so others can benefit. I don't take freelance jobs.
Ik lees geen PM -> als je iets te vragen of te melden hebt, doe het op het forum, zodat anderen er ook wat aan hebben.


Re: [CutListEditor] Crash when called again #24 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 26 October 2021 - 06:24

@Erik Slagter - here are two created 'problematic' .cuts files

 

 

How is possible in most cases simulate on H7 wrong .cuts file: 

- remove .cuts file

- open movie in CutListEditor

- skip to some time, use STOP for pause movie, then severals use "left/right" and set "remove before this position".

- exit editor

- open same movie again in CutListEditor ... it should crash, if not, try again similar steps

 

but on vs1500 is impossible for me create wrong .cuts file (but edit movie with 'corrupted' .cuts from H7 crash on vs1500 too)

 

Attached Files

  • Attached File  cuts.zip   323bytes   2 downloads

Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #25 Dimitrij

  • PLi® Core member
  • 10,258 posts

+347
Excellent

Posted 26 October 2021 - 16:32

CutListEditor: Improvements


GigaBlue UHD Quad 4K /Lunix3-4K/Duo 4K


Re: [CutListEditor] Crash when called again #26 WanWizard

  • PLi® Core member
  • 70,204 posts

+1,798
Excellent

Posted 26 October 2021 - 18:07

Are they relevant? The first one looks cosmetic.


Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)

Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.

Many answers to your question can be found in our new and improved wiki.


Re: [CutListEditor] Crash when called again #27 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 2 January 2022 - 13:25

I put before 'ASSERTION' in filepush.cpp debug message:

eDebug("[* * * eFilePushThread * * *] current_span_remaining: %d m_blocksize: %d", current_span_remaining, m_blocksize);  

And with problematic cuts file I got:

[* * * eFilePushThread * * *] current_span_remaining: 195484520 m_blocksize: 188
[eDVBServicePlay] have 1 video stream(s)../../git/lib/dvb/filepush.cpp:72 ASSERTION !(current_span_remaining % m_blocksize) FAILED!

Edited by ims, 2 January 2022 - 13:26.

Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #28 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 23 January 2022 - 02:00

It seems, it could solve this problem


Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #29 nautilus7

  • Senior Member
  • 229 posts

+6
Neutral

Posted 23 January 2022 - 03:49

Hi, that PR is for the python3 branch... Nothing to do with the issue reported here.

Edited by nautilus7, 23 January 2022 - 03:52.


Re: [CutListEditor] Crash when called again #30 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 23 January 2022 - 08:41

I know, that it is pr for python3, but for python2 there should be same for writing cutlist to binary .cuts, imho. I tried it and it seems, crashing dissappeared for me now.


Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #31 Dimitrij

  • PLi® Core member
  • 10,258 posts

+347
Excellent

Posted 23 January 2022 - 10:29

ims

Maybe need apply this https://github.com/O...5ceb4e6f9a04594


Edited by Dimitrij, 23 January 2022 - 10:29.

GigaBlue UHD Quad 4K /Lunix3-4K/Duo 4K


Re: [CutListEditor] Crash when called again #32 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 23 January 2022 - 11:13

Writing cutlist as binary it solve, imho ... 

No crash more....

 

I tried write cutlist twice to 2 files with "old" code (1,2) and you can see differences in size on attached screenshot in .cuts and .cuts1, cuts2.

 

PR is here

Attached Files

  • Attached File  bin.PNG   22.48KB   0 downloads

Edited by ims, 23 January 2022 - 11:18.

Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #33 Dimitrij

  • PLi® Core member
  • 10,258 posts

+347
Excellent

Posted 23 January 2022 - 11:58

Great news.


GigaBlue UHD Quad 4K /Lunix3-4K/Duo 4K


Re: [CutListEditor] Crash when called again #34 nautilus7

  • Senior Member
  • 229 posts

+6
Neutral

Posted 23 January 2022 - 12:30

@ims I doubt your PR will change anything on this matter. Apart that python 2 works with bytes by default, the crash described here is possibly related to the cpp code.

Re: [CutListEditor] Crash when called again #35 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 23 January 2022 - 13:25

agree ... but weird, that used "reset" on movie write same cutlist with different size...


Edited by ims, 23 January 2022 - 13:26.

Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #36 WanWizard

  • PLi® Core member
  • 70,204 posts

+1,798
Excellent

Posted 23 January 2022 - 15:00

In which case it is not much use to merge the PR?


Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)

Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.

Many answers to your question can be found in our new and improved wiki.


Re: [CutListEditor] Crash when called again #37 ims

  • PLi® Core member
  • 13,764 posts

+214
Excellent

Posted 23 January 2022 - 15:12

In which case it is not much use to merge the PR?

pr closed


Kdo nic nedělá, nic nezkazí!

Re: [CutListEditor] Crash when called again #38 WanWizard

  • PLi® Core member
  • 70,204 posts

+1,798
Excellent

Posted 22 December 2022 - 19:02

The filepump requires fixed records, 188 bytes long, and expects remaining size to be an exact multiple of 188.

 

I'm looking in the cutlisteditor code, but nowhere is there a check on this size, whatever is the current position seems to be saved.

As the assert() in the filepump suggests the record size is a prerequisite for processing, I'd say the cutlisteditor has to make sure the data written in the cuelist should reflect that, but if I look at the code, all mark points are in time, not in bytes.

 

Which means dvb.cpp returns a non-aligned value somewhere, after seeking through the recording for a mark point in time.


Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)

Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.

Many answers to your question can be found in our new and improved wiki.


Re: [CutListEditor] Crash when called again #39 WanWizard

  • PLi® Core member
  • 70,204 posts

+1,798
Excellent

Posted 22 December 2022 - 22:34

Bug found, fixed, and backported to 8.3-release.


Currently in use: VU+ Duo 4K (2xFBC S2), VU+ Solo 4K (1xFBC S2), uClan Usytm 4K Ultimate (S2+T2), Octagon SF8008 (S2+T2), Zgemma H9.2H (S2+T2)

Due to my bad health, I will not be very active at times and may be slow to respond. I will not read the forum or PM on a regular basis.

Many answers to your question can be found in our new and improved wiki.


Re: [CutListEditor] Crash when called again #40 Qu@rk

  • Senior Member
  • 192 posts

+2
Neutral

Posted 23 December 2022 - 08:03

Thank you. Merry Christmas and happy holidays! 




3 user(s) are reading this topic

0 members, 3 guests, 0 anonymous users