From 47b9db5941ba11277d7bd62060057202236fb921 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Wed, 28 Jan 2015 23:49:19 +0000
Subject: [PATCH] regress: Support NULL fdset arguments to select

We are going to want to pass these, and currently the test harness
will dereference the NULL.  So fix this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---
 regress/hcommon.c      | 4 ++++
 regress/hcommon.c.m4   | 5 +++++
 regress/hplayback.c    | 4 ++++
 regress/hplayback.c.m4 | 5 +++++
 4 files changed, 18 insertions(+)

diff --git a/regress/hcommon.c b/regress/hcommon.c
index e63a8cd..301fb2f 100644
--- a/regress/hcommon.c
+++ b/regress/hcommon.c
@@ -167,6 +167,10 @@ void Tvbbytes(const void *buf, int len) {
 void Tvbfdset(int max, const fd_set *fds) {
   int i;
   const char *comma= "";
+  if (!fds) {
+    Tvba("null");
+    return;
+  }
   Tvba("[");
   for (i=0; i<max; i++) {
     if (!FD_ISSET(i,fds)) continue;
diff --git a/regress/hcommon.c.m4 b/regress/hcommon.c.m4
index 330da4d..760ca0c 100644
--- a/regress/hcommon.c.m4
+++ b/regress/hcommon.c.m4
@@ -160,6 +160,11 @@ void Tvbfdset(int max, const fd_set *fds) {
   int i;
   const char *comma= "";
   
+  if (!fds) {
+    Tvba("null");
+    return;
+  }
+
   Tvba("[");
   for (i=0; i<max; i++) {
     if (!FD_ISSET(i,fds)) continue;
diff --git a/regress/hplayback.c b/regress/hplayback.c
index 279934c..ed23542 100644
--- a/regress/hplayback.c
+++ b/regress/hplayback.c
@@ -102,6 +102,10 @@ static void P_updatetime(void) {
 static void Pfdset(fd_set *set, int max) {
   int r, c;
   char *ep;
+  if (!set) {
+    Pstring("null","null fdset pointer");
+    return;
+  }
   if (vb2.buf[vb2.used++] != '[') Psyntax("fd set start not [");
   FD_ZERO(set);
   if (vb2.buf[vb2.used] == ']') { vb2.used++; return; }
diff --git a/regress/hplayback.c.m4 b/regress/hplayback.c.m4
index 475144a..792952f 100644
--- a/regress/hplayback.c.m4
+++ b/regress/hplayback.c.m4
@@ -150,6 +150,11 @@ static void P_updatetime(void) {
 static void Pfdset(fd_set *set, int max) {
   int r, c;
   char *ep;
+
+  if (!set) {
+    Pstring("null","null fdset pointer");
+    return;
+  }
   
   if (vb2.buf[vb2.used++] != hm_squote[hm_squote) Psyntax("fd set start not [");
   FD_ZERO(set);
-- 
2.11.0