diff options
author | Dan Fandrich <dan@coneharvesters.com> | 2008-11-02 05:01:39 +0000 |
---|---|---|
committer | Dan Fandrich <dan@coneharvesters.com> | 2008-11-02 05:01:39 +0000 |
commit | 484d549ecef3ee44dff4bbca418b5ace20629502 (patch) | |
tree | b2c366b79b716fd53dd4461b4839666ea8920e79 /lib/sendf.c | |
parent | d098ab436d458735ed16dfcc6ab43cd4574bac57 (diff) |
Marked with TODO comments a number of problems in the Kerberos code detected
while investigating the issue in http://curl.haxx.se/mail/lib-2008-09/0262.html
I'm hesitant to fix them because I have no way of testing the result.
Diffstat (limited to 'lib/sendf.c')
-rw-r--r-- | lib/sendf.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/sendf.c b/lib/sendf.c index 7a8c3cad2..4fe0b4806 100644 --- a/lib/sendf.c +++ b/lib/sendf.c @@ -630,13 +630,16 @@ int Curl_read(struct connectdata *conn, /* connection data */ return -1; #endif if(nread < 0) - /* since it is negative and not EGAIN, it was a protocol-layer error */ + /* since it is negative and not EAGAIN, it was a protocol-layer error */ return CURLE_RECV_ERROR; } else { if(conn->sec_complete) nread = Curl_sec_read(conn, sockfd, buffertofill, bytesfromsocket); + /* TODO: Need to handle EAGAIN here somehow, similar to how it + * is done in Curl_read_plain, either right here or in Curl_sec_read + * itself. */ else { int ret = Curl_read_plain(sockfd, buffertofill, bytesfromsocket, &nread); |