valgrind: check successful read before attempting write

... this is actually not really a bug, but I can see why valgrind complains, so let's keep it happy.

valgrind: check successful read before attempting write
... this is actually not really a bug, but I can see why valgrind complains, so let's keep it happy.
84894459 · Wesley W. Terpstra · 10a8bcc8 · 84894459
Commit 84894459 authored Aug 31, 2015 by Wesley W. Terpstra
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 5 deletions

dev.c api/transport/dev.c +7 -5

No files found.
--- a/api/transport/dev.c
+++ b/api/transport/dev.c
@@ -94,11 +94,13 @@ eb_status_t eb_dev_connect(struct eb_transport* transportp, struct eb_link* link
  link->fdes = fdes;
  link->flags = fcntl(fdes, F_GETFL, 0);

-  tcgetattr(fdes, &ios);
-  cfmakeraw(&ios);
-  cfsetispeed(&ios, B115200);
-  cfsetospeed(&ios, B115200);
-  tcsetattr(fdes, TCSANOW, &ios);
+  // If this is a serial device, enter raw mode
+  if (tcgetattr(fdes, &ios) == 0) {
+    cfmakeraw(&ios);
+    cfsetispeed(&ios, B115200);
+    cfsetospeed(&ios, B115200);
+    tcsetattr(fdes, TCSANOW, &ios);
+  }
  
  eb_dev_set_blocking(link, 0);
  /* Discard any data unread by last user */