SourceForge has been redesigned. Learn more.
Close

#171 Crash due to forgotten null check in affixmgr.cpp

closed-fixed
None
5
2010-12-20
2010-12-20
Anonymous
No

case 'I': { if (TESTAFF(rv->astr, 'J', rv->alen)) numsyllable += 1; break; }

probably ought to be

case 'I': { if (rv && TESTAFF(rv->astr, 'J', rv->alen)) numsyllable += 1; break; }

The same snippet appears twice in the file, but the null check was forgotten in one case (the wonders of code duplication ;)

Discussion

  • Németh László

    • assigned_to: nobody --> nemethl
    • status: open --> closed-fixed
     
  • Németh László

    Fixed in the CVS. Thanks, László