# HG changeset patch # User Christian Hammond # Date 1076104865 0 # Node ID dcace041cfb84228cd31c55416d51600ff25957e # Parent 83830cb8467eff4ad038aff2e5b2c3b5d48cab4b [gaim-migrate @ 8939] It's an uncommon practice to use strlen("SOMETHING") && strncmp(...) as the value for a length parameter to a strncmp. I'm therefore assuming it's wrong, and fixing it. I'm also going to get some breakfast. I was thinking Lucky Charms, but we're about out of that, so I may go for bacon instead. Or I could go take my shower and get some donuts. It's a tough decision. committer: Tailor Script diff -r 83830cb8467e -r dcace041cfb8 src/protocols/yahoo/yahoo.c --- a/src/protocols/yahoo/yahoo.c Fri Feb 06 20:43:55 2004 +0000 +++ b/src/protocols/yahoo/yahoo.c Fri Feb 06 22:01:05 2004 +0000 @@ -1922,8 +1922,8 @@ GString *s; len = read(source, buf, sizeof(buf)-1); - if (len <= 0 || (strncmp(buf, "HTTP/1.0 302", strlen("HTTP/1.0 302") && - strncmp(buf, "HTTP/1.1 302", strlen("HTTP/1.1 302"))))) { + if (len <= 0 || (strncmp(buf, "HTTP/1.0 302", strlen("HTTP/1.0 302")) && + strncmp(buf, "HTTP/1.1 302", strlen("HTTP/1.1 302")))) { gaim_connection_error(gc, _("Unable to read")); return; }