diff options
| author | Marcus Meissner <marcus@jet.franken.de> | 2016-11-19 21:57:47 +0100 |
|---|---|---|
| committer | Marcus Meissner <marcus@jet.franken.de> | 2016-11-19 21:57:47 +0100 |
| commit | 98a7e4e5a4fb4371f5802d505089cb7bb962eed0 (patch) | |
| tree | e8c8a8b760736042add03a34caa882ddd7018197 | |
| parent | 458880a1034950a9a4a72fe9f8896c621602aedc (diff) | |
| download | libgphoto2-98a7e4e5a4fb4371f5802d505089cb7bb962eed0.tar.gz | |
start is unsigned (Coverity)
| -rw-r--r-- | camlibs/ptp2/config.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/camlibs/ptp2/config.c b/camlibs/ptp2/config.c index d0f429c76..810829b4e 100644 --- a/camlibs/ptp2/config.c +++ b/camlibs/ptp2/config.c @@ -5014,7 +5014,10 @@ _get_SONY_BatteryLevel(CONFIG_GET_ARGS) { if (dpd->FormFlag == PTP_DPFF_Range) { gp_widget_set_name (*widget, menu->name); start = dpd->FORM.Range.MinimumValue.i8; - if (start == -1) start = 0; /* -1 might be special for unknown? */ + if (dpd->FORM.Range.MinimumValue.i8 == -1) + start = 0; /* -1 might be special for unknown? */ + else + start = dpd->FORM.Range.MinimumValue.i8; end = dpd->FORM.Range.MaximumValue.i8; value_float = dpd->CurrentValue.i8; if (0 == end - start + 1) { |