c - Linux socket send() error EAGAIN -


i designing http tunnel using proxy , http connect request. 90% of time, when try send connect request connect 192.168.1.68:3001 http/1.1\r\n\r\n send fails, returning errno = 11, "try again".

i using non-blocking sockets, , socket closed re-opened between connect attempts. don't think send buffer full, because not sending data. i'm confused why connecting , not (and there doesn't appear pattern, connect twice in row, there 10 failures between connects).

what of reasons getting error?

edit:

(code)

if (connect(sock, psockaddr, isockaddrsize)) {     ...     char message[80];     sprintf(message, "connect 192.168.1.68:3001 http/1.1\r\n\r\n");     printf("*j* message loaded.\n"); //*j*     printf("*j* %s\n", message); //*j*     if (send(sock, message, strlen(message) + 1, 0) < 0) printf("*j* send failed: %d\n", errno);      else printf("*j* data sent.\n");     } ... printf("\n*j* close socket (reset host command)"); closesocket(sock);

(output)

[success] *j* starting connect... *j* message loaded. *j* connect 192.168.1.68:3001 http/1.1   *j* data sent. *j* rv = 1 *j* ending connect...  [fail] *j* starting connect... *j* message loaded. *j* connect 192.168.1.68:3001 http/1.1   *j* send failed: 11 *j* ending connect...

you have several bugs in code:

  1. connect() returns 0 on success , -1 on error. if statement evaluates true non-zero value. means code sending connect message when connect() failing error. however, not errors fatal. in particular, since using non-blocking socket, einprogress/wsaewouldblock error indicates connection still pending , need use select() or (e)poll() wait socket finish connecting (or fail). so, code work if connect() connects asynchronously , happens connect proxy before have chance call send(). behavior should not rely on. need handle errors correctly.

  2. the return value of send() specifies how many bytes accepted, or -1 on error. fewer bytes requested may accepted (especially on non-blocking socket). , need handle non-fatal errors, in particular (wsa)ewouldblock (and eagain, , possibly eintr, on non-windows platforms), means socket unable accept new data @ moment send() called, call send() again. should calling send() in loop until bytes have been sent, or fatal error occurs.

  3. even if send()'ing data correctly, use of + 1 after strlen() wrong. including null terminator in data sent, not part of connect protocol. null terminator treated proxy application data , forwarded as-is through tunnel next server (if tunnel opened), breaking communications server.

with said, try more following instead. use of closeocket() indicates programming windows, windows-specific:

int ret, err; ... ret = connect(sock, psockaddr, isockaddrsize); if (ret == -1) {     err = wsagetlasterror();      if (err == wsaewouldblock)     {         fd_set wfd;         fd_zero(&wfd);         fd_set(sock, &wfd);          fd_set efd;         fd_zero(&efd);         fd_set(sock, &efd);          timeval timeout;         timeout.tv_sec = ...;         timeout.tv_usec = ...;          ret = select(0, null, &wfd, &wfd, &timeout);          if (ret == 0)         {             printf("*j* connect timeout\n");              // handle timeout needed ...             return;         }          if ((ret > 0) && fd_isset(sock, &efd))         {             err = 0;             getsockopt(sock, sol_socket, so_error, (char*)&err, sizeof(err));             ret = -1;         }     }      if (ret == -1)     {         printf("*j* connect failed: %d\n", err);          // handle fatal error needed ...         return;     } }  ...  char message[80]; sprintf(message, "connect 192.168.1.68:3001 http/1.1\r\n\r\n"); printf("*j* message loaded.\n"); //*j* printf("*j* %s\n", message); //*j*  char *pmsg = message; int len = strlen(message);  {     ret = send(sock, pmsg, len, 0);     if (ret == -1)     {         err = getlastsocketerror();              if (err == wsaewouldblock)         {             fd_set wfd;             fd_zero(&wfd);             fd_set(sock, &wfd);              timeval timeout;             timeout.tv_sec = ...;             timeout.tv_usec = ...;              ret = select(0, null, &wfd, null, &timeout);              if (ret > 0)                 continue;              if (ret == 0)             {                 printf("*j* send timeout\n");                  // handle timeout needed ...                 return;             }              err = wsagetlasterror();         }          printf("*j* send failed: %d\n", err);          // handle fatal error needed ...         return;     }     else     {         pmsg += ret;         len += ret;     } } while (len > 0);  printf("*j* data sent\n");  ...  printf("*j* close socket\n"); closesocket(sock);

if want little more cross-platform, try instead (or use cross-platform socket library instead):

int getlastsocketerror() {     #ifdef windows     return wsagetlasterror();     #else     return errno;     #endif };  int ret, err; ... ret = connect(sock, psockaddr, isockaddrsize); if (ret == -1) {     err = getlastsocketerror();          if (         #ifdef windows         err == wsaewouldblock         #else         err == einprogress         #endif         )     {         #ifndef windows         {         #endif          fd_set wfd;         fd_zero(&wfd);         fd_set(sock, &wfd);          #ifdef windows         fd_set efd;         fd_zero(&efd);         fd_set(sock, &efd);         #endif          timeval timeout;         timeout.tv_sec = ...;         timeout.tv_usec = ...;          #ifdef windows         ret = select(0, null, &wfd, &wfd, &timeout);         #else         ret = select(sock+1, null, &wfd, null, &timeout);         #endif          if (ret == 0)         {             printf("*j* connect timeout\n");              // handle timeout needed ...             return;         }          #ifdef windows         if ((ret > 0) && fd_isset(sock, &efd))         {             err = 0;             getsockopt(sock, sol_socket, so_error, (char*)&err, sizeof(err));             ret = -1;         }         #endif          #ifndef windows         } while ((ret == -1) && (errno == eintr));         #endif     }      if (ret == -1)     {         printf("*j* connect failed: %d\n", err);          // handle fatal error needed ...         return;     } }  ...  char message[80]; sprintf(message, "connect 192.168.1.68:3001 http/1.1\r\n\r\n"); printf("*j* message loaded.\n"); //*j* printf("*j* %s\n", message); //*j*  char *pmsg = message; int len = strlen(message);  {     ret = send(sock, pmsg, len, 0);     if (ret == -1)     {         err = getlastsocketerror();              if (             #ifdef windows             err == wsaewouldblock             #else             (err == ewouldblock) || (err == eagain)             #endif             )         {             fd_set wfd;             fd_zero(&wfd);             fd_set(sock, &wfd);              timeval timeout;             timeout.tv_sec = ...;             timeout.tv_usec = ...;              ret = select(                 #ifdef windows                 0                 #else                 sock+1                 #endif                 , null, &wfd, null, &timeout);              if (ret > 0)                 continue;              if (ret == 0)             {                 printf("*j* send timeout\n");                  // handle timeout needed ...                 return;             }              err = getlastsocketerror();         }          #ifndef windows         if (err != eintr)         #endif         {             printf("*j* send failed: %d\n", err);              // handle fatal error needed ...             return;         }     }     else     {         pmsg += ret;         len += ret;     } } while (len > 0);  printf("*j* data sent\n");  ...  printf("*j* close socket\n");  #ifdef windows closesocket(sock); #else close(sock); #endif

Comments

Post a Comment

Popular posts from this blog

Is there a better way to structure post methods in Class Based Views -

i'm using class based views in django , never specify url in ajax calls , omit action parameter in form because know these requests go through post method of class based view. notice view - profile page make many requests depending on user updating on page - post method become convoluted , if/else mess example: def post(self, request, *args, **kwargs): # handle user changing available date if request.post.get('availabledate') != none: self.updateavailabledate(request) return jsonresponse({'result':'success'}) if request.post.get('newprojectname') != none: creative_user = creativeuserprofile.objects.get(id = request.user.id) project = project.create_project(creative_user, request.post['newprojectname']) project.save() return jsonresponse({'projectid': project.id }) if request.post.get('projectimage') != none , request.files['file']: proj...
Read more

performance - Why is XCHG reg, reg a 3 micro-op instruction on modern Intel architectures? -

i'm doing micro-optimization on performance critical part of code , came across sequence of instructions (in at&t syntax): add %rax, %rbx mov %rdx, %rax mov %rbx, %rdx i thought had use case xchg allow me shave instruction , write: add %rbx, %rax xchg %rax, %rdx however, dimay found agner fog's instruction tables , xchg 3 micro-op instruction 2 cycle latency on sandy bridge, ivy bridge, broadwell, haswell , skylake. 3 whole micro-ops , 2 cycles of latency! 3 micro-ops throws off 4-1-1-1 cadence , 2 cycle latency makes worse original in best case since last 2 instructions in original might execute in parallel. now... cpu might breaking instruction micro-ops equivalent to: mov %rax, %tmp mov %rdx, %rax mov %tmp, %rdx where tmp anonymous internal register , suppose last 2 micro-ops run in parallel latency 2 cycles. given register renaming occurs on these micro-architectures, though, doesn't make sense me done way. why wouldn't register renamer ...
Read more

c# - Asp.net web api : redirect unauthorized requst to forbidden page -

im trying redirect unauthorized request forbidden page instead i'm getting forbidden page in response body , how can fix ? here's startup class : app.createperowincontext(storecontext.create); app.createperowincontext<applicationusermanager>(applicationusermanager.create); app.createperowincontext<applicationsigninmanager>(applicationsigninmanager.create); app.usecookieauthentication(new cookieauthenticationoptions { authenticationtype = defaultauthenticationtypes.applicationcookie, expiretimespan = timespan.fromdays(30), }); app.useexternalsignincookie(defaultauthenticationtypes.externalcookie); app.usetwofactorsignincookie(defaultauthenticationtypes.twofactorcookie, timespan.fromminutes(5)); app.usetwofactorrememberbrowsercookie(defaultauthenticationtypes.twofactorrememberbrowsercookie); the method im trying reach : [httpget] [authorize(roles = "admin")] public string getcurrentusername() { return userman...
Read more