act_wiz.c in function do_advance around line 2949 find

         for( sn = 0; sn <= MAX_SKILL; sn++ )

change that to

         for( sn = 0; sn < MAX_SKILL; sn++ )

Remcon said:

act_wiz.c in function do_advance around line 2949 find gd

         for( sn = 0; sn <= MAX_SKILL; sn++ )

change that to

         for( sn = 0; sn < MAX_SKILL; sn++ )

Using < MAX_SKILL instead of <= MAX_SKILL should prevent the loop from running one extra iteration when sn reaches the maximum value.

Has this change already been tested in all cases, especially when sn is exactly equal to MAX_SKILL?