1
0
mirror of https://github.com/mpv-player/mpv synced 2024-12-23 23:32:26 +00:00

stream_dvb: global protection mutex and usage bit for global_dvb_state.

The mutex is used in dvbin_open and dvbin_close only since these are
the only entry / exit points to the stream.
When opening, it is first checked (mutexed) whether the state already exists
and is in use, then a STREAM_ERROR is returned,
since there may be only one stream_dvb active at a time.
State-creation itself is also protected by mutex.
In dvbin_close, the usage-bit is set to false (mutexed) in case
of channel switch.
In case of stream-teardown, the state is destructed
(also protected by mutex).
This commit is contained in:
Oliver Freyermuth 2016-01-08 18:37:30 +01:00 committed by wm4
parent 1e46bda0f8
commit c55b242b4e
2 changed files with 21 additions and 1 deletions

View File

@ -103,6 +103,7 @@ typedef struct {
int timeout; int timeout;
int last_freq; int last_freq;
bool switching_channel; bool switching_channel;
bool stream_used;
} dvb_state_t; } dvb_state_t;
typedef struct dvb_params { typedef struct dvb_params {

View File

@ -40,6 +40,7 @@
#include <unistd.h> #include <unistd.h>
#include <fcntl.h> #include <fcntl.h>
#include <errno.h> #include <errno.h>
#include <pthread.h>
#include <libavutil/avstring.h> #include <libavutil/avstring.h>
@ -63,6 +64,7 @@
#define OPT_BASE_STRUCT struct dvb_params #define OPT_BASE_STRUCT struct dvb_params
static dvb_state_t* global_dvb_state = NULL; static dvb_state_t* global_dvb_state = NULL;
static pthread_mutex_t global_dvb_state_lock = PTHREAD_MUTEX_INITIALIZER;
/// URL definition /// URL definition
static const m_option_t stream_params[] = { static const m_option_t stream_params[] = {
@ -820,9 +822,13 @@ static void dvbin_close(stream_t *stream)
{ {
dvb_priv_t *priv = (dvb_priv_t *) stream->priv; dvb_priv_t *priv = (dvb_priv_t *) stream->priv;
dvb_state_t* state = priv->state; dvb_state_t* state = priv->state;
if (state->switching_channel && state->is_on) { if (state->switching_channel && state->is_on) {
// Prevent state destruction, reset channel-switch. // Prevent state destruction, reset channel-switch.
state->switching_channel = false; state->switching_channel = false;
pthread_mutex_lock(&global_dvb_state_lock);
global_dvb_state->stream_used = false;
pthread_mutex_unlock(&global_dvb_state_lock);
return; return;
} }
@ -838,8 +844,11 @@ static void dvbin_close(stream_t *stream)
state->fe_fd = state->dvr_fd = -1; state->fe_fd = state->dvr_fd = -1;
state->is_on = 0; state->is_on = 0;
pthread_mutex_lock(&global_dvb_state_lock);
dvb_free_state(state); dvb_free_state(state);
global_dvb_state = NULL; global_dvb_state = NULL;
pthread_mutex_unlock(&global_dvb_state_lock);
} }
static int dvb_streaming_start(stream_t *stream, int tuner_type, char *progname) static int dvb_streaming_start(stream_t *stream, int tuner_type, char *progname)
@ -897,7 +906,17 @@ static int dvb_open(stream_t *stream)
char *progname; char *progname;
int tuner_type = 0, i; int tuner_type = 0, i;
pthread_mutex_lock(&global_dvb_state_lock);
if (global_dvb_state && global_dvb_state->stream_used) {
MP_ERR(stream, "DVB stream already in use, only one DVB stream can exist at a time!");
pthread_mutex_unlock(&global_dvb_state_lock);
return STREAM_ERROR;
}
dvb_state_t* state = dvb_get_state(stream); dvb_state_t* state = dvb_get_state(stream);
state->stream_used = true;
pthread_mutex_unlock(&global_dvb_state_lock);
priv->state = state; priv->state = state;
if (state == NULL) { if (state == NULL) {
MP_ERR(stream, "DVB CONFIGURATION IS EMPTY, exit\n"); MP_ERR(stream, "DVB CONFIGURATION IS EMPTY, exit\n");
@ -982,6 +1001,7 @@ dvb_state_t *dvb_get_state(stream_t *stream)
state->count = 0; state->count = 0;
state->switching_channel = false; state->switching_channel = false;
state->stream_used = true;
state->cards = NULL; state->cards = NULL;
state->fe_fd = state->dvr_fd = -1; state->fe_fd = state->dvr_fd = -1;
for (int i = 0; i < MAX_CARDS; i++) { for (int i = 0; i < MAX_CARDS; i++) {
@ -1073,7 +1093,6 @@ dvb_state_t *dvb_get_state(stream_t *stream)
} }
global_dvb_state = state; global_dvb_state = state;
return state; return state;
} }