[Debichem-devel] Bug#703965: update and fix

fulvio ciriaco oivulf at gmail.com
Tue Mar 26 14:24:07 UTC 2013


Dear maintainer,
I discovered the bug is still there in ver. 2.4.6, much after
the ver. 2.4.3 in unstable.
I partially fixed the bug and filed the bug upstream.
The bug is due to code in AnimationGeomConv ~line 1444,
the following is the fix:
/* sprintf(listOfAtoms[j].symbol,"%s",AtomCoord[0]); */
/* sprintf(listOfAtoms[j].mmType,"%s",AtomCoord[0]); */
/* sprintf(listOfAtoms[j].pdbType,"%s",AtomCoord[0]); */
g_strlcpy(listOfAtoms[j].symbol,AtomCoord[0],5);
g_strlcpy(listOfAtoms[j].mmType,AtomCoord[0],5);
g_strlcpy(listOfAtoms[j].pdbType,AtomCoord[0],5);
i.e. comment the sprintf calls in the small strings
symbol mmtype and pdbtype and truncate the parsed names
into them. g_strlcpy is in glib, and doesn't add any dependency,
I think.
This in not completely satisfactory, the string length should
be increased and there are other problems I notified to the
developer, but at least it makes the software usable.
Thanks
Fulvio Ciriaco



More information about the Debichem-devel mailing list