-rw-r--r-- | libopie2/opiecore/device/odevice_palm.cpp | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/libopie2/opiecore/device/odevice_palm.cpp b/libopie2/opiecore/device/odevice_palm.cpp index 399c08c..d40dfce 100644 --- a/libopie2/opiecore/device/odevice_palm.cpp +++ b/libopie2/opiecore/device/odevice_palm.cpp | |||
@@ -231,2 +231,3 @@ int Palm::displayBrightnessResolution() const | |||
231 | int res = 1; | 231 | int res = 1; |
232 | int fd = -1; | ||
232 | 233 | ||
@@ -237,7 +238,8 @@ int Palm::displayBrightnessResolution() const | |||
237 | case Model_Palm_Z72: | 238 | case Model_Palm_Z72: |
238 | int fd = ::open( m_backlightdev + "max_brightness", O_RDONLY|O_NONBLOCK ); | 239 | fd = ::open( m_backlightdev + "max_brightness", O_RDONLY|O_NONBLOCK ); |
239 | if ( fd ) | 240 | if ( fd != -1) |
240 | { | 241 | { |
241 | char buf[100]; | 242 | char buf[100]; |
242 | if ( ::read( fd, &buf[0], sizeof buf ) ) ::sscanf( &buf[0], "%d", &res ); | 243 | if ( ::read( fd, &buf[0], sizeof buf ) > 0 ) |
244 | ::sscanf( &buf[0], "%d", &res ); | ||
243 | ::close( fd ); | 245 | ::close( fd ); |
@@ -263,2 +265,3 @@ bool Palm::setDisplayBrightness( int bright ) | |||
263 | int val = ( bright == 1 ) ? 1 : ( bright * numberOfSteps ) / 255; | 265 | int val = ( bright == 1 ) ? 1 : ( bright * numberOfSteps ) / 255; |
266 | int fd = -1; | ||
264 | 267 | ||
@@ -270,4 +273,4 @@ bool Palm::setDisplayBrightness( int bright ) | |||
270 | case Model_Palm_Z72: | 273 | case Model_Palm_Z72: |
271 | int fd = ::open( m_backlightdev + "brightness", O_WRONLY|O_NONBLOCK ); | 274 | fd = ::open( m_backlightdev + "brightness", O_WRONLY|O_NONBLOCK ); |
272 | if ( fd ) | 275 | if ( fd != -1 ) |
273 | { | 276 | { |
@@ -280,3 +283,4 @@ bool Palm::setDisplayBrightness( int bright ) | |||
280 | 283 | ||
281 | default: res = false; | 284 | default: |
285 | res = false; | ||
282 | } | 286 | } |