From 50a2e734db093829f974324f58c78592f3ee8536 Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Mon, 10 Apr 2000 19:27:29 +0000 Subject: fix various bugs in APOP code (still untested) and some of the error * providers/pop3/camel-pop3-store.c (pop3_connect): fix various bugs in APOP code (still untested) and some of the error cases. svn path=/trunk/; revision=2376 --- camel/ChangeLog | 3 ++ camel/providers/pop3/camel-pop3-store.c | 52 +++++++++++++++++++-------------- 2 files changed, 33 insertions(+), 22 deletions(-) diff --git a/camel/ChangeLog b/camel/ChangeLog index db125eaf11..984cc4b81b 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,5 +1,8 @@ 2000-04-10 Dan Winship + * providers/pop3/camel-pop3-store.c (pop3_connect): fix various + bugs in APOP code (still untested) and some of the error cases. + * camel-provider.h: Clarify what provider.protocol, provider.name, and provider.description should be. diff --git a/camel/providers/pop3/camel-pop3-store.c b/camel/providers/pop3/camel-pop3-store.c index 8afc77fd47..3713c086a6 100644 --- a/camel/providers/pop3/camel-pop3-store.c +++ b/camel/providers/pop3/camel-pop3-store.c @@ -210,13 +210,10 @@ pop3_connect (CamelService *service, CamelException *ex) { struct hostent *h; struct sockaddr_in sin; - int fd, status, apoplen; - char *buf, *apoptime; + int fd, status; + char *buf, *apoptime, *apopend; CamelPop3Store *store = CAMEL_POP3_STORE (service); - if (!service_class->connect (service, ex)) - return FALSE; - h = camel_service_gethost (service, ex); if (!h) return FALSE; @@ -256,46 +253,54 @@ pop3_connect (CamelService *service, CamelException *ex) /* Read the greeting, note APOP timestamp, if any. */ buf = camel_stream_buffer_read_line (CAMEL_STREAM_BUFFER (store->istream)); - if (!buf) - return -1; + if (!buf) { + camel_exception_set (ex, CAMEL_EXCEPTION_SERVICE_UNAVAILABLE, + "Could not read greeting from POP " + "server."); + return FALSE; + camel_stream_close (store->ostream); + camel_stream_close (store->istream); + } apoptime = strchr (buf, '<'); - if (apoptime) { - int len = strcspn (apoptime, ">"); - - apoptime = g_strndup (apoptime, len); - } else + apopend = apoptime ? strchr (apoptime, '>') : NULL; + if (apoptime && apopend) + apoptime = g_strndup (apoptime, apopend - apoptime + 1); + else apoptime = NULL; g_free (buf); /* Authenticate via APOP if we can, USER/PASS if we can't. */ status = CAMEL_POP3_FAIL; - if (apoptime && apoptime[apoplen] == '>') { + if (apoptime) { char *secret, md5asc[32], *d; unsigned char md5sum[16], *s; - secret = g_strdup_printf("%.*s%s", apoplen + 1, apoptime, - service->url->passwd); - md5_get_digest(secret, strlen(secret), md5sum); - g_free(secret); + secret = g_strdup_printf ("%s%s", apoptime, + service->url->passwd); + md5_get_digest (secret, strlen (secret), md5sum); + g_free (apoptime); + g_free (secret); for (s = md5sum, d = md5asc; d < md5asc + 32; s++, d += 2) - sprintf(d, "%.2x", *s); + sprintf (d, "%.2x", *s); - status = camel_pop3_command(store, NULL, "APOP %s %s", - service->url->user, md5asc); + status = camel_pop3_command (store, NULL, "APOP %s %s", + service->url->user, md5asc); } if (status != CAMEL_POP3_OK ) { char *msg; - status = camel_pop3_command(store, &msg, "USER %s", - service->url->user); + status = camel_pop3_command (store, &msg, "USER %s", + service->url->user); if (status != CAMEL_POP3_OK) { camel_exception_setv (ex, CAMEL_EXCEPTION_SERVICE_CANT_AUTHENTICATE, "Unable to connect to POP " "server. Error sending username:" " %s", msg ? msg : "(Unknown)"); g_free (msg); + camel_stream_close (store->ostream); + camel_stream_close (store->istream); return FALSE; } @@ -307,10 +312,13 @@ pop3_connect (CamelService *service, CamelException *ex) "server. Error sending password:" " %s", msg ? msg : "(Unknown)"); g_free (msg); + camel_stream_close (store->ostream); + camel_stream_close (store->istream); return FALSE; } } + service_class->connect (service, ex); return TRUE; } -- cgit