Re: another string length problem

On 3 Jul 2007, at 3:24:49 PM, Benoit Grégoire wrote:

> On Monday 25 June 2007, David Reiser wrote:
>> ameritrade seems to have added some new proprietary tags that cause
>> gnucash 2.x to crash in ofx_preproc.cpp similar to another string
>> issue fixed last fall.
>>
>> Christian Stimming sent me the attached patch to test, and it does
>> keep gnucash from crashing while importing the offending file.
>
> I'm not sure the patch is correct.  I suspect it just masks the  
> issue.  Can
> you send the test case that caused the crash?
>
> -- 
> Benoit Grégoire
> Technologies Coeus inc.

Attached is a short ofx file that crashes libofx 0.8.3  (under  
gnucash 2.1.x). Importing the file into the same version of gnucash  
does not crash if the libofx patch from cstim (proptagfix.patch in my  
june 25 email) has been applied to libofx. The indenting is mine (not  
Ameritrade's). The full file from which this is excerpted does not  
have the indenting. The behavior of the full file is the same -  
crashes existing libofx, does not crash with the patch.

My later musings regarding CUSIPs for the sweep account proved to be  
incorrect. Ameritrade did provide an entry in <seclist> early this  
year, but did not in my June downloads. The presence or absence of  
the sweep account in the <seclist> does not affect this crash.

Dave

--
David Reiser
[email protected]

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

_______________________________________________
Libofx-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libofx-devel