libECBUFR

decoded value of missing numeric with scaling not as missing but as upper limit value of descriptor

Bug #994015 reported by vanh souvanlasy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libECBUFR
Fix Committed
High
vanh souvanlasy
libECBUFR 0.8.5

Bug Description

In decoding of a BUFR message, when the value of a descriptor is defined with scaling (float) and has the value of 'MISSING',
the decoder is reporting a value corresponding to the value obtained when all bits is on, which is the maximum possible
value of a descriptor. Which is normally a value out of range for this descriptor.

vanh souvanlasy (vanh-souvanlasy)
Changed in libecbufr:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → vanh souvanlasy (vanh-souvanlasy)
status: Confirmed → In Progress
Revision history for this message
vanh souvanlasy (vanh-souvanlasy) wrote :

after the value is decoded, it is checked if it is missing, but the unchecked value was passed instead of the checked one.

Changed in libecbufr:
status: In Progress → Fix Committed
importance: Critical → High
Revision history for this message
cpb (chris-beauregard) wrote :

Vanh, r240 breaks the regression test on Test/BUFR/dpbm_fostats.bufr:

diff BUFR/dpbm_fostats.out BUFR/dpbm_fostats.out.tmp
60c60
< 010002 MSNG
---
> 010002 VALUE=42949672960.0
 93,94c93,94
< 002153 {R=0} MSNG
< 002154 {R=0} MSNG
---
> 002153 {R=0} VALUE=429496729600000000.0
> 002154 {R=0} VALUE=429496729600000000.0
 99,100c99,100
< 002153 {R=0} MSNG
< 002154 {R=0} MSNG
...

vanh souvanlasy (vanh-souvanlasy)
Changed in libecbufr:
status: Fix Committed → In Progress
vanh souvanlasy (vanh-souvanlasy)
Changed in libecbufr:
status: In Progress → Fix Committed
Revision history for this message
vanh souvanlasy (vanh-souvanlasy) wrote :

reverted changes made, added missing value test to bufr_cvt_i64_dval() .
Also, added test to bufr_is_missing_float() and bufr_is_missing_double() for infinite values to be considered as missing.

Revision history for this message
cpb (chris-beauregard) wrote :

Thanks, that's got it...

vanh souvanlasy (vanh-souvanlasy)
Changed in libecbufr:
milestone: none → 0.8.5
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.