Commit f8774ff611f32395ff6a6958d21fbbd245961688
2012-06-26T04:14:57
Log a specific error when serial opens fail due to lack of user permissions
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45
diff --git a/fpgautils.c b/fpgautils.c
index 39f3abd..fd5f8d0 100644
--- a/fpgautils.c
+++ b/fpgautils.c
@@ -15,6 +15,7 @@
#include <string.h>
#ifndef WIN32
+#include <errno.h>
#include <termios.h>
#include <sys/stat.h>
#include <unistd.h>
@@ -157,7 +158,20 @@ serial_open(const char*devpath, unsigned long baud, signed short timeout, bool p
#ifdef WIN32
HANDLE hSerial = CreateFile(devpath, GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, NULL);
if (unlikely(hSerial == INVALID_HANDLE_VALUE))
+ {
+ DWORD e = GetLastError();
+ switch (e) {
+ case ERROR_ACCESS_DENIED:
+ applog(LOG_ERR, "Do not have user privileges required to open %s", devpath);
+ break;
+ case ERROR_SHARING_VIOLATION:
+ applog(LOG_ERR, "%s is already in use by another process", devpath);
+ break;
+ default:
+ break;
+ }
return -1;
+ }
// thanks to af_newbie for pointers about this
COMMCONFIG comCfg = {0};
@@ -188,7 +202,11 @@ serial_open(const char*devpath, unsigned long baud, signed short timeout, bool p
int fdDev = open(devpath, O_RDWR | O_CLOEXEC | O_NOCTTY);
if (unlikely(fdDev == -1))
+ {
+ if (errno == EACCES)
+ applog(LOG_ERR, "Do not have user privileges required to open %s", devpath);
return -1;
+ }
struct termios pattr;
tcgetattr(fdDev, &pattr);