67c4256f7e
Daniel Stenberg
2025-10-13
Subject
pop3: function could get the ->transfer field wrong
Binary
No binary (build failed)
Architecture & Compiler Flags
—
Build Log
(log yok)
Commit Diff GitHub →
lib/pop3.c+6 -7
| @@ -1377,18 +1377,17 @@ static CURLcode pop3_perform(struct Curl_easy *data, bool *connected, | |
| DEBUGF(infof(data, "DO phase starts")); | |
| - | if(data->req.no_body) { |
| + | /* Start the first command in the DO phase, may alter data->req.no_body */ |
| + | result = pop3_perform_command(data); |
| + | if(result) |
| + | return result; |
| + | |
| + | if(data->req.no_body) |
| /* Requested no body means no transfer */ | |
| pop3->transfer = PPTRANSFER_INFO; | |
| - | } |
| *dophase_done = FALSE; /* not done yet */ | |
| - | /* Start the first command in the DO phase */ |
| - | result = pop3_perform_command(data); |
| - | if(result) |
| - | return result; |
| - | |
| /* Run the state-machine */ | |
| result = pop3_multi_statemach(data, dophase_done); | |
| *connected = Curl_conn_is_connected(data->conn, FIRSTSOCKET); | |