关于最新的ReadTouch工程的一个小BUG，请证

发布于 2012-11-14 19:54:50 浏览：2796 订阅该版

最新的Readtouch工程和以前的工程里面在http.c文件中 
```
static int http_connect(struct http_session* session,
    struct sockaddr_in * server, char *host_addr, const char *url)
{
	int socket_handle;
	int peer_handle;
	int rc;
	char mimeBuffer[100];
	int timeout = HTTP_RCV_TIMEO;

if((socket_handle = socket( PF_INET, SOCK_STREAM, IPPROTO_TCP )) < 0)
	{
		rt_kprintf( "HTTP: SOCKET FAILED
" );
		return -1;
	}

/* set recv timeout option */
	setsockopt(socket_handle, SOL_SOCKET, SO_RCVTIMEO, (void*)&timeout, sizeof(timeout));

peer_handle = connect( socket_handle, (struct sockaddr *) server, sizeof(*server));
	if ( peer_handle < 0 )
	{
		rt_kprintf( "HTTP: CONNECT FAILED %i
", peer_handle );
		return -1;
	}

{
		char *buf;
		rt_uint32_t length;

buf = rt_malloc (512);
		if (*url)
			length = rt_snprintf(buf, 512, _http_get, url, host_addr);
		else
			length = rt_snprintf(buf, 512, _http_get, "/", host_addr);

rc = send(peer_handle, buf, length, 0);
		// rt_kprintf("HTTP request:
%s", buf);

/* release buffer */
		rt_free(buf);
	}

// We now need to read the header information
	while ( 1 )
	{
		int i;

// read a line from the header information.
		rc = http_read_line( peer_handle, mimeBuffer, 100 );
		// rt_kprintf(">> %s
", mimeBuffer);

if ( rc < 0 ) return rc;

// End of headers is a blank line.  exit.
		if (rc == 0) break;
		if ((rc == 2) && (mimeBuffer[0] == '
')) break;

// Convert mimeBuffer to upper case, so we can do string comps
		for(i = 0; i < strlen(mimeBuffer); i++)
			mimeBuffer* = toupper(mimeBuffer*);

if(strstr(mimeBuffer, "HTTP/1.")) // First line of header, contains status code. Check for an error code
		{
			rc = http_is_error_header(mimeBuffer);
			if(rc)
			{
				rt_kprintf("HTTP: status code = %d!
", rc);
				return -rc;
			}
		}

if(strstr(mimeBuffer, "CONTENT-LENGTH:"))
		{
			session->size = http_parse_content_length(mimeBuffer);
			rt_kprintf("size = %d
", session->size);
		}
	}

// We've sent the request, and read the headers.  SockHandle is
	// now at the start of the main data read for a file io read.
	return peer_handle;
}
```
此函数中的
```
peer_handle = connect( socket_handle, (struct sockaddr *) server, sizeof(*server));
rc = send(peer_handle, buf, length, 0);
rc = http_read_line( peer_handle, mimeBuffer, 100 );
return peer_handle;
```
是否应该改为
```
rc = send( socket_handle, buf, length, 0);
rc = http_read_line( socket_handle, mimeBuffer, 100 );
return socket_handle; 
```
因为我查看资料反应connect的返回值只是告诉连接是否成功。而不是返回套接字。。套接字是第一个参数，而后面的发送函数。接收函数。返回socket都是使用了connect的返回值。而不是使用第一个参数（套接字），是否需要修改?