Andy Ross <[EMAIL PROTECTED]> said:

> This is what I did last night.  It works for the bug I was seeing, but
> I didn't go through the code looking for other printf's that might
> blow up with the same values.  The hard-coded "8" should be unified
> with the static definition of SliderText, of course.
> 
> As Norman and I pointed out, though, a better solution would probably
> be to sanify the values before use if possible.  Sometimes, if they
> come from arbitrary user input, you can't and have to resort to buffer
> size checking with snprintf and the like.  But since these are angles,
> it should be possible.
> 

That's basically what I did to get YASim models to load reliably on my system.
 I changed the FGInterface::_calc_timespan() (? guessing name from memory) so
that it is sanifying delta_time to non-negative and under 10 seconds.  But
there's a bug somewhere that is causing these wild values to show up
occasionally in the first place.  I'm convinced that a couple of the reports
we've had lately of trouble that miraculously goes away when some applies a
seemingly unrelated CVS update are due to data corruption.

Best,

Jim

_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel

Reply via email to