Menu ▾ ▴

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

closed-fixed
Németh László
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ó

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

  • Németh László

    Németh László - 2010-12-20

    Fixed in the CVS. Thanks, László