Curtis L. Olson writes:
> You bastards! Writing property names to char arrays that are too short for
> the data you are putting in it. :-( Fixed ...
> for (index = 0; index < MAX_ENGINES; index++) {
> ! char name[32];
> sprintf(name, "/controls/engines/engine[%d]/throttle", index);
> fgTie(name, this, index,
> --- 276,280 ----
>
> for (index = 0; index < MAX_ENGINES; index++) {
> ! char name[MAX_NAME_LEN];
> sprintf(name, "/controls/engines/engine[%d]/throttle", index);
> fgTie(name, this, index,
etc.
Note that there is no need to code things this way. Here's a cleaner
alternative, avoiding sprintf altogether:
for (index = 0; index < MAX_ENGINES; index++) {
SGPropertyNode * node =
fgGetNode("/controls/engines/engine", index, true);
node->tie("throttle",
SGRawValueMethodsIndexed(this, index,
&FGControls::get_throttle,
&FGControls::set_throttle));
node->tie("mixture",
SGRawValueMethodsIndexed(this, index,
&FGControls::get_mixture,
&FGControls::set_mixture));
// and so on ...
}
All the best,
David
--
David Megginson, [EMAIL PROTECTED], http://www.megginson.com/
_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
