The grpnam variable was declared as an array of 9 chars, when it only needed to be 6. There was very robust code to set up the solid, region, and group names that I shortened with simple strncpy's.
Not a functional change. Patch should have used bu_strlcpy and should not have been broken up into multiple trackers, just increases review/apply burden (for an already trivial patch).
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
mk_bolt.c
Logged In: YES
user_id=785737
Originator: NO
Not a functional change. Patch should have used bu_strlcpy and should not have been broken up into multiple trackers, just increases review/apply burden (for an already trivial patch).