Alfons Hoogervorst ([info]alfons) wrote,
@ 2004-01-24 22:06:00
Previous Entry  Add to memories!  Tell a Friend  Next Entry
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.



Advertisement


(No comments)

Post a comment in response:

From:
( )Anonymous - this user has disabled anonymous and non-friend posting. You may post here if alfons lists you as a friend.
Help(will be screened)
Identity URL: 
(will be screened)
Username:
Password:
Don't have an account? Create one now.
Subject:
No HTML allowed in subject
   Help
Message:
 
Notice! This user has turned on the option that logs your IP address when posting. Help
Create an Account
Forgot your login or password?
Login w/ OpenID
English • Español • Deutsch • Русский…