summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Hendricks <david.hendricks@gmail.com>2020-07-02 09:36:50 -0700
committerAngel Pons <th3fanbus@gmail.com>2020-07-07 14:02:02 +0000
commit174a0c1b405281158d32b457e5652cc961260549 (patch)
treed99d1a417605a1a954ddebf63bc93d7a3e3433cb
parent9b6fb745c2945725d0c6d5b2f17991dde9bb39b0 (diff)
downloadflashrom-174a0c1b405281158d32b457e5652cc961260549.tar.gz
flashrom-174a0c1b405281158d32b457e5652cc961260549.tar.bz2
flashrom-174a0c1b405281158d32b457e5652cc961260549.zip
serial: Fix file read/write error handling for Windows
File read/write semantics are different between POSIX and Windows. In particular Windows file read/write functions return a boolean type to indicate success or failure, while the POSIX equivalents return a signed integer indicating number of bytes read if successful or -1 if not. This attempts to correct some error handling paths for Windows and avoid invalid comparisons that were causing compilation issues. Reported on https://github.com/flashrom/flashrom/issues/149 Change-Id: Ib179d51ede2dbd38f54f3641bfe90340a6a87e31 Signed-off-by: David Hendricks <david.hendricks@gmail.com> Reviewed-on: https://review.coreboot.org/c/flashrom/+/43051 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--serial.c30
1 files changed, 22 insertions, 8 deletions
diff --git a/serial.c b/serial.c
index 3a99dbff1..76d34a263 100644
--- a/serial.c
+++ b/serial.c
@@ -391,14 +391,17 @@ int serialport_write(const unsigned char *buf, unsigned int writecnt)
while (writecnt > 0) {
#if IS_WINDOWS
- WriteFile(sp_fd, buf, writecnt, &tmp, NULL);
+ if (!WriteFile(sp_fd, buf, writecnt, &tmp, NULL)) {
+ msg_perr("Serial port write error!\n");
+ return 1;
+ }
#else
tmp = write(sp_fd, buf, writecnt);
-#endif
if (tmp == -1) {
msg_perr("Serial port write error!\n");
return 1;
}
+#endif
if (!tmp) {
msg_pdbg2("Empty write\n");
empty_writes--;
@@ -425,14 +428,17 @@ int serialport_read(unsigned char *buf, unsigned int readcnt)
while (readcnt > 0) {
#if IS_WINDOWS
- ReadFile(sp_fd, buf, readcnt, &tmp, NULL);
+ if (!ReadFile(sp_fd, buf, readcnt, &tmp, NULL)) {
+ msg_perr("Serial port read error!\n");
+ return 1;
+ }
#else
tmp = read(sp_fd, buf, readcnt);
-#endif
if (tmp == -1) {
msg_perr("Serial port read error!\n");
return 1;
}
+#endif
if (!tmp)
msg_pdbg2("Empty read\n");
readcnt -= tmp;
@@ -485,17 +491,21 @@ int serialport_read_nonblock(unsigned char *c, unsigned int readcnt, unsigned in
for (i = 0; i < timeout; i++) {
msg_pspew("readcnt %u rd_bytes %u\n", readcnt, rd_bytes);
#if IS_WINDOWS
- ReadFile(sp_fd, c + rd_bytes, readcnt - rd_bytes, &rv, NULL);
+ if (!ReadFile(sp_fd, c + rd_bytes, readcnt - rd_bytes, &rv, NULL)) {
+ msg_perr_strerror("Serial port read error: ");
+ ret = -1;
+ break;
+ }
msg_pspew("read %lu bytes\n", rv);
#else
rv = read(sp_fd, c + rd_bytes, readcnt - rd_bytes);
msg_pspew("read %zd bytes\n", rv);
-#endif
if ((rv == -1) && (errno != EAGAIN)) {
msg_perr_strerror("Serial port read error: ");
ret = -1;
break;
}
+#endif
if (rv > 0)
rd_bytes += rv;
if (rd_bytes == readcnt) {
@@ -565,17 +575,21 @@ int serialport_write_nonblock(const unsigned char *buf, unsigned int writecnt, u
for (i = 0; i < timeout; i++) {
msg_pspew("writecnt %u wr_bytes %u\n", writecnt, wr_bytes);
#if IS_WINDOWS
- WriteFile(sp_fd, buf + wr_bytes, writecnt - wr_bytes, &rv, NULL);
+ if (!WriteFile(sp_fd, buf + wr_bytes, writecnt - wr_bytes, &rv, NULL)) {
+ msg_perr_strerror("Serial port write error: ");
+ ret = -1;
+ break;
+ }
msg_pspew("wrote %lu bytes\n", rv);
#else
rv = write(sp_fd, buf + wr_bytes, writecnt - wr_bytes);
msg_pspew("wrote %zd bytes\n", rv);
-#endif
if ((rv == -1) && (errno != EAGAIN)) {
msg_perr_strerror("Serial port write error: ");
ret = -1;
break;
}
+#endif
if (rv > 0) {
wr_bytes += rv;
if (wr_bytes == writecnt) {