Skip to content

Commit fb4420e

Browse files
committed
compress_done: add assertion
`./bin/hd-rum-transcode 8M 5004 100::` seem to be crashing due to wrong state (assert it)`
1 parent 2ff0bd5 commit fb4420e

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

src/video_compress.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
*/
4040

4141
#include <cassert>
42-
#include <cinttypes>
42+
#include <cstdint> // for uint32_t
4343
#include <cstdio>
4444
#include <cstring>
4545
#include <memory>
@@ -51,6 +51,7 @@
5151
#include "messaging.h"
5252
#include "module.h"
5353
#include "tv.h"
54+
#include "utils/macros.h" // for to_fourcc
5455
#include "utils/misc.h" // for format_number_with_delim
5556
#include "utils/synchronized_queue.h"
5657
#include "utils/thread.h"
@@ -61,6 +62,7 @@
6162
#include "lib_common.h"
6263
#include "debug.h"
6364

65+
constexpr uint32_t MAGIC = to_fourcc('v','c','m','p');
6466
#define MOD_NAME "[vcompress] "
6567

6668
using namespace std;
@@ -109,6 +111,7 @@ struct compress_state {
109111
}
110112
~compress_state() { module_done(&mod); }
111113

114+
uint32_t magic = MAGIC;
112115
struct module mod; ///< compress module data
113116
struct compress_state_real *ptr{}; ///< pointer to real compress state
114117
synchronized_queue<shared_ptr<video_frame>, 1> queue;
@@ -495,6 +498,7 @@ compress_done(struct compress_state *proxy)
495498
if (proxy == nullptr) {
496499
return;
497500
}
501+
assert(proxy->magic == MAGIC);
498502

499503
struct compress_state_real *s = proxy->ptr;
500504
if (!proxy->poisoned) { // pass poisoned pill if it wasn't

0 commit comments

Comments
 (0)