Fix invalid memory usage in mpegts/iptv, mpegts/tsfile, subscriptions…

…, and tvhtime

src/input/mpegts/iptv/iptv_rtsp.c

@@ -197,12 +197,14 @@ iptv_rtsp_stop
   ( iptv_mux_t *im )
 {
   rtsp_priv_t *rp = im->im_data;
-  int play = rp->play;
+  int play;
 
   lock_assert(&global_lock);
 
   if (rp == NULL)
     return;
+
+  play = rp->play;
   im->im_data = NULL;
   rp->hc->hc_aux = NULL;
   if (play)

src/input/mpegts/tsfile/tsfile.c

@@ -47,14 +47,17 @@ tsfile_network_create_service
 {
   pthread_mutex_lock(&tsfile_lock);
   mpegts_service_t *s = mpegts_service_create1(NULL, mm, sid, pmt_pid, NULL);
-  s->s_config_save = tsfile_service_config_save;
-  pthread_mutex_unlock(&tsfile_lock);
 
   // TODO: HACK: REMOVE ME
   if (s) {
+    s->s_config_save = tsfile_service_config_save;
+    pthread_mutex_unlock(&tsfile_lock);
     channel_t *c = channel_create(NULL, NULL, NULL);
     service_mapper_link((service_t*)s, c, NULL);
   }
+  else
+    pthread_mutex_unlock(&tsfile_lock);
+
   return s;
 }
 

src/subscriptions.c

@@ -549,13 +549,15 @@ subscription_input(void *opauqe, streaming_message_t *sm)
 void
 subscription_unsubscribe(th_subscription_t *s, int quiet)
 {
-  service_t *t = s->ths_service;
+  service_t *t;
   char buf[512];
   size_t l = 0;
 
   if (s == NULL)
     return;
 
+  t = s->ths_service;
+
   lock_assert(&global_lock);
 
   s->ths_state = SUBSCRIPTION_ZOMBIE;

src/tvhtime.c

@@ -67,8 +67,9 @@ ntp_shm_init ( void )
     return NULL;
 
   shmptr = shmat(shmid, 0, 0);
-  memset(shmptr, 0, sizeof(ntp_shm_t));
+
   if (shmptr) {
+    memset(shmptr, 0, sizeof(ntp_shm_t));
     shmptr->mode      = 1;
     shmptr->precision = -1;
     shmptr->nsamples  = 1;