laforge submitted this change.

Approvals:
  Jenkins Builder: Verified
  daniel: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved

when traffic dump directory is specified, now prints error message and exits instead of crashing when traffic file cannot be written

Change-Id: I273e07aefd9faea13662f6699a6f3b074c8efe88
---
M src/lower_mac/tetra_lower_mac.c
1 file changed, 5 insertions(+), 0 deletions(-)

diff --git a/src/lower_mac/tetra_lower_mac.c b/src/lower_mac/tetra_lower_mac.c
index ca1b72e..f6a893a 100644
--- a/src/lower_mac/tetra_lower_mac.c
+++ b/src/lower_mac/tetra_lower_mac.c
@@ -22,6 +22,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <errno.h>
 #include <linux/limits.h>
 
 #include <osmocom/core/utils.h>
@@ -196,6 +197,10 @@
 		snprintf(fname, sizeof(fname), "%s/traffic_%d_%d.out", tms->dumpdir,
 			tms->cur_burst.is_traffic, tms->tsn);
 		f = fopen(fname, "ab");
+		if (!f) {
+			fprintf(stderr, "Could not open dump file %s for writing: %s\n", fname, strerror(errno));
+			exit(1);
+		}
 
 		/* Generate a block */
 		memset(block, 0x00, sizeof(int16_t) * 690);

To view, visit change 28926. To unsubscribe, or for help writing mail filters, visit settings.