Try fix for Cutlist editor to restore missing .ap and .sc files.

Two separate bugs were introduced in 5.6.003 with the SoftCSA changes:

  Bug 1 — lib/dvb/pvrparse.cpp

  processPacket() was changed to return -2 for a "broken startcode" (was return 0 in 5.6.002). The new parseData() propagates this via early exit, and
  asyncWrite() treats -2 as stream corruption and calls return len before the actual disk write — meaning recording data is silently discarded. This
  affects channels where a PUSI packet's PES payload doesn't begin with 0x000001 (CI-decrypted channels, MPEG-1 audio, subtitle streams, etc.).

  Fix: revert broken startcode to return 0. A missing PES start code means "skip PTS extraction for this packet" — it is not stream corruption.

  Bug 2 — lib/dvb/filepush.cpp (the intermittent ~1/3 cause)

  eFilePushThreadRecorder::m_protocol was never initialized in the constructor. 5.6.003 added a !getProtocol() guard before calling
  m_ts_parser.parseData() in asyncWrite(). When the recorder object is allocated from previously-used heap memory, m_protocol contains garbage from a
  prior session. !getProtocol() then returns false, parseData() is never called, no PTS timestamps are collected, and stopSave() finds no access points
  and skips writing .ap entirely.

  This explains the intermittent pattern: the first recording after a reboot uses fresh zero-initialized memory (works), later recordings reuse freed
  heap (fails ~1/3 of the time depending on what the allocator returns).

  Fix: add m_protocol(0), m_session_id(0), m_stream_id(0), m_packet_no(0) to the constructor initializer list in filepush.cpp.

Thx @the bandit

Author: Ev0-BH

lib/dvb/filepush.cpp

@@ -345,7 +345,11 @@ eFilePushThreadRecorder::eFilePushThreadRecorder(unsigned char* buffer, size_t b
 	m_stop(1),
 	m_buffer_fill(0),
 	m_buffer_min_write(0),
-	m_messagepump(eApp, 0, "eFilePushThreadRecorder")
+	m_messagepump(eApp, 0, "eFilePushThreadRecorder"),
+	m_protocol(0),
+	m_session_id(0),
+	m_stream_id(0),
+	m_packet_no(0)
 {
 	CONNECT(m_messagepump.recv_msg, eFilePushThreadRecorder::recvEvent);
 

lib/dvb/pvrparse.cpp

@@ -922,7 +922,14 @@ int eMPEGStreamParserTS::processPacket(const unsigned char *pkt, off_t offset)
 			//}
 			//eDebugNoNewLine("\n");
 
-			return -2;
+			/* Return 0 (not -2): a missing/invalid PES start code means we
+			 * cannot extract a PTS from this packet, but the stream is not
+			 * corrupt — CI-decrypted streams, MPEG-1 audio, and similar
+			 * sources regularly produce PUSI packets without a 0x000001
+			 * start code.  Returning -2 caused parseData() to abort early
+			 * and asyncWrite() to skip writing data to disk, resulting in
+			 * empty recordings and missing .ap/.sc files. */
+			return 0;
 		}
 
 		if (pkt[7] & 0x80) // PTS present?