another round of coverity fixes

diff --git a/src/dvr/dvr_db.c b/src/dvr/dvr_db.c
index e4b844ecfb6ef5b83e967220f84e59427d464ba5..e13107b2a5793db3b4739a712f93174eec58a59c 100644 (file)
--- a/src/dvr/dvr_db.c
+++ b/src/dvr/dvr_db.c
@@ -1679,7 +1679,7 @@ dvr_entry_update
 void 
 dvr_event_replaced(epg_broadcast_t *e, epg_broadcast_t *new_e)
 {
-  dvr_entry_t *de;
+  dvr_entry_t *de, *de_next;
   channel_t *ch = e->channel;
   epg_broadcast_t *e2;
   char ubuf[UUID_HEX_SIZE];
@@ -1691,7 +1691,8 @@ dvr_event_replaced(epg_broadcast_t *e, epg_broadcast_t *new_e)
   if (ch == NULL || e == new_e) return;
 
   /* Existing entry */
-  LIST_FOREACH(de, &ch->ch_dvrs, de_channel_link) {
+  for (de = LIST_FIRST(&ch->ch_dvrs); de; de = de_next) {
+    de_next = LIST_NEXT(de, de_channel_link);
 
     if (de->de_bcast != e)
       continue;

diff --git a/src/epggrab/module/psip.c b/src/epggrab/module/psip.c
index e2029f73bc98872ba247bf9115358f1636d7380c..4622e10f788f633440847b190d67c576d4d3a7fb 100644 (file)
--- a/src/epggrab/module/psip.c
+++ b/src/epggrab/module/psip.c
@@ -368,6 +368,9 @@ _psip_eit_callback_channel
     length = (ptr[6] & 0x0f) << 16 | ptr[7] << 8 | ptr[8];
     stop = start + length;
     titlelen = ptr[9];
+
+    if (12 + titlelen > len) break;
+
     dlen = ((ptr[10+titlelen] & 0x0f) << 8) | ptr[11+titlelen];
     size = titlelen + dlen + 12;
     tvhtrace("psip", "  %03d: titlelen %d, dlen %d", i, titlelen, dlen);

diff --git a/src/wizard.c b/src/wizard.c
index 89dee714431a7d4c862c2b3f739e4132b91e9ac6..f49a6aded6d47e6cc48f059edffcc0ed4e0f5c2a 100644 (file)
--- a/src/wizard.c
+++ b/src/wizard.c
@@ -334,7 +334,7 @@ static void login_save(idnode_t *in)
     }
     htsmsg_destroy(conf);
 
-    if (s && s[0] != '*' && w->password && w->password[0]) {
+    if (s[0] != '*' && w->password[0]) {
       conf = htsmsg_create_map();
       htsmsg_add_bool(conf, "enabled", 1);
       htsmsg_add_str(conf, "username", s);