diff options
| author | Elliott Hughes | 2015-04-28 19:24:24 -0500 |
|---|---|---|
| committer | Elliott Hughes | 2015-04-29 19:46:43 -0500 |
| commit | 7bad7c4646ee8fd8d6e6ed0ffd3ddbb0c1b41a2f (patch) | |
| tree | e2c357667b0e76e6fa5a43c7a7951c872c373e9c /ui.cpp | |
| parent | 8c389e9c769c8632cf16c0794f1e84f49996077c (diff) | |
| download | platform-bootable-recovery-7bad7c4646ee8fd8d6e6ed0ffd3ddbb0c1b41a2f.tar.gz platform-bootable-recovery-7bad7c4646ee8fd8d6e6ed0ffd3ddbb0c1b41a2f.tar.xz platform-bootable-recovery-7bad7c4646ee8fd8d6e6ed0ffd3ddbb0c1b41a2f.zip | |
Check all lseek calls succeed.
Also add missing TEMP_FAILURE_RETRYs on read, write, and lseek.
Bug: http://b/20625546
Change-Id: I03b198e11c1921b35518ee2dd005a7cfcf4fd94b
Diffstat (limited to 'ui.cpp')
| -rw-r--r-- | ui.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
| @@ -251,7 +251,7 @@ bool RecoveryUI::IsUsbConnected() { | |||
| 251 | 251 | ||
| 252 | char buf; | 252 | char buf; |
| 253 | // USB is connected if android_usb state is CONNECTED or CONFIGURED. | 253 | // USB is connected if android_usb state is CONNECTED or CONFIGURED. |
| 254 | int connected = (read(fd, &buf, 1) == 1) && (buf == 'C'); | 254 | int connected = (TEMP_FAILURE_RETRY(read(fd, &buf, 1)) == 1) && (buf == 'C'); |
| 255 | if (close(fd) < 0) { | 255 | if (close(fd) < 0) { |
| 256 | printf("failed to close /sys/class/android_usb/android0/state: %s\n", | 256 | printf("failed to close /sys/class/android_usb/android0/state: %s\n", |
| 257 | strerror(errno)); | 257 | strerror(errno)); |