Home

Advertisement

Customize
Jan. 24th, 2004 @ 10:06 pm Waiting for SF CVS

Waiting for SF CVS, but at least I can connect and cvs update, though it is quite slow.
I also fixed yet another bug in my local copy: bug report, "sylpheed 0.9.8claws42 crashes on some messages." The problem is a NULL pointer dereference in procmime.c. Offending code in bold:


  g_strstrip(hentry[5].body);

  if ((hentry[5].body != NULL) &&

    (sscanf(hentry[5].body, "%d.%d", &mime_major, &mime_minor) == 2) &&

    (mime_major == 1) && (mime_minor == 0)) {

Actually, the bug could have been prevented if the developer knew how the sscanf() function works: The g_strstrip(), to remove a string's leading and trailing white space, is not necessary because the sscanf() function discards whitespaces in the input string.

About this Entry