Bugs item #3428428, was opened at 2011-10-25 23:27
Message generated for change (Tracker Item Submitted) made by dalke
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=428740&aid=3428428&group_id=40728

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Stereochemistry
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Andrew Dalke (dalke)
Assigned to: Nobody/Anonymous (nobody)
Summary: incomplete check for @SP chiral class

Initial Comment:
The current parser checks for the "@S" of the "@SP" chiral class and assumes 
the next character will be a "P".

This means OpenBabel allows the invalid SMILES of "F[Po@S92](I)(Br)Cl"

This may also trigger a possible read error/segfault on a string like "[C@S" 
because it will skip the NUL terminator and try to process the succeeding 
characters. Worst case is if that happens to be in unallocated memory, which is 
unlikely.

I regard this as a low-priority task.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=428740&aid=3428428&group_id=40728

------------------------------------------------------------------------------
The demand for IT networking professionals continues to grow, and the
demand for specialized networking skills is growing even more rapidly.
Take a complimentary Learning@Cisco Self-Assessment and learn 
about Cisco certifications, training, and career opportunities. 
http://p.sf.net/sfu/cisco-dev2dev
_______________________________________________
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Reply via email to