11 Aug 2003, Konstantin Olchanski, , Alarm on no ping? 
 | 
I want midas alarms to go off when I cannot ping arbitrary remote hosts. Is
 
there is easy/preferred way to do this? K.O. | 
   
   10 Oct 2003, Konstantin Olchanski, , mhttpd crash on corrupted ODB /RunInfo 
 | 
There was no feedback. This code has been commited. K.O.
 
 
> Invalid values of ODB /RunInfo/State cause mhttpd crash in
 | 
   12 Oct 2003, Konstantin Olchanski, , Array overruns in mhttpd.c::submit_elog() 
 | 
While adding new functionality to submit_elog() (add the message text to the
 
outgoing email), I noticed that the email text is being stored into an array
 
of size 256, mail_text[256], without any checks for array overrun. This
 | 
   
   12 Oct 2003, Konstantin Olchanski, , Array overruns in mhttpd.c::submit_elog() 
 | 
> While adding new functionality to submit_elog() (add the message text to the
 
> outgoing email), I noticed that the email text is being stored into an array
 
> of size 256, mail_text[256], without any checks for array overrun. This
 | 
   12 Oct 2003, Konstantin Olchanski, , mhttpd: add Elog text to outgoing email. 
 | 
This commit adds the elog message text to the outgoing email message. This
 
functionality has been requested a logn time ago, but I guess nobody got
 
around to implement it, until now. I also added assert() traps for the most
 | 
   12 Oct 2003, Konstantin Olchanski, , Refuse to set run number zero 
 | 
I am debugging the frequent problem where the run number is mysteriously
 
reset to zero. As a first step, I am commiting changes to mhttpd.c and midas.c:
 
- abort on obviously corrupted "run number < 0"
 | 
   
   12 Oct 2003, Konstantin Olchanski, , Refuse to set run number zero 
 | 
> I am debugging the frequent problem where the run number is mysteriously
 
> reset to zero. As a first step, I am commiting changes to mhttpd.c and midas.c:
 
> - abort on obviously corrupted "run number < 0"
 | 
   
   13 Oct 2003, Konstantin Olchanski, , mhttpd: add Elog text to outgoing email. 
 | 
> > around to implement it, until now. I also added assert() traps for the most
 
> > common array overruns in the Elog code.
 
> 
 | 
   
   13 Oct 2003, Konstantin Olchanski, , Array overruns in mhttpd.c::submit_elog() 
 | 
> > > While adding new functionality to submit_elog() ....
 
> 
 
> The whole elog functionality in mhttpd will be replaced (sometime) ...
 | 
   15 Oct 2003, Konstantin Olchanski, , test 
 | 
test
 
test
 
test | 
   
   15 Oct 2003, Konstantin Olchanski, , test 
 | 
> test
 
> test
 
> test
 | 
   31 Oct 2003, Konstantin Olchanski, , mana.c without ROOT and HBOOK 
 | 
Stephan, why did you prohibit building mana.c without ROOT and HBOOK
 
support? I think such a configuration is valid and should be allowed.
 
 | 
   31 Oct 2003, Konstantin Olchanski, , Disable "tab"s in xemacs 
 | 
The default C indentation style in xemacs uses "tab" characters, violating
 
the MIDAS coding convention. To disable this misfeature in xemacs (emacs
 
too?), put this incantation in your .xemacs/custom.el file:
 | 
   31 Oct 2003, Konstantin Olchanski, , more odb "run number" error checking 
 | 
I added error checking to the places where we read "/runinfo/run number". In
 
general, I do this:
 
 | 
   31 Oct 2003, Konstantin Olchanski, , Do not frob "/runinfo" in mhttpd.c 
 | 
I found where we tickle the race condition in db_create_record().
 
 
1) in mhttpd.c,  every time we show the status page, we call
 | 
   
   01 Nov 2003, Konstantin Olchanski, , more odb  
 | 
> > I added error checking to the places where we read "/runinfo/run number". 
 
> Now YOU broke the system by editing all these files with something I consider 
 
> temporary debugging code. A run number of zero is *VALILD*.
 | 
   
   01 Nov 2003, Konstantin Olchanski, , mana.c without ROOT and HBOOK 
 | 
> > Stephan, why did you prohibit building mana.c without ROOT and HBOOK
 
> > support? I think such a configuration is valid and should be allowed.
 
> 
 | 
   
   01 Nov 2003, Konstantin Olchanski, , Do not frob  
 | 
> > I found where we tickle the race condition in db_create_record().
 
> The reason for the db_create_record() is the following: Assume that we change 
 
> the /runinfo structure...
 | 
   15 Nov 2003, Konstantin Olchanski, , Phantom "open records" 
 | 
Sometimes (maybe after a client uncleanly exits?), I see phantom "open
 
records", for example:
 
[local:twist:Running]Gas>sor
 | 
   20 Nov 2003, Konstantin Olchanski, , midas timeout wraparound 
 | 
While reviving midas on midtig01 after it was not used for a while, we see
 
this. Notice negative "last called" numbers. Looks like a time_t wraparound
 
somewhere...
 |