Back Midas Rome Roody Rootana
  Midas DAQ System  Not logged in ELOG logo
Entry  13 Feb 2008, Konstantin Olchanski, Info, mhttpd history display updates 
    Reply  14 Feb 2008, Stefan Ritt, Info, mhttpd history display updates 
       Reply  21 Feb 2008, Konstantin Olchanski, Info, mhttpd history display updates 
Message ID: 443     Entry time: 21 Feb 2008     In reply to: 432
Author: Konstantin Olchanski 
Topic: Info 
Subject: mhttpd history display updates 
> You misspelled one ODB entry:
> Line 9014:
>             sprintf(str, "/History/Display/%s/Label", path);
> 
> Line 9028:
>             sprintf(str, "/History/Display/%s/Labels", path);
>                                                 ---^
> 
> I wonder how you could have tested that code for 1/2 year without noticing this error.
> I fixed and committed it.


It turns out that the program was tested as originally committed. With the above
modification, it corrupts ODB - originally, it used the wrong array element size to create
the wrong array. Corrected, it creates the right array with the wrong size, then
subsequent db_set_data_index() happily corrupts ODB.

Fix for mhttpd committed as svn revision 4128.
Fix for ODB corruption committed at svn revision 4129 (also fixes extract_key())


K.O.
ELOG V3.1.4-2e1708b5