X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=http_recv.c;h=c418af129a0560fdb09e2aa2cf32e1c7b68146ed;hp=a681a003dadf83f4d8ce3763f794e5215e3a8de1;hb=75375ddad82067c8f5cf3ecafc3dd231eeb393b0;hpb=f23be61d476e3f354e31072da0e0e77050a8dc56

diff --git a/http_recv.c b/http_recv.c
index a681a003..c418af12 100644
--- a/http_recv.c
+++ b/http_recv.c
@@ -87,7 +87,7 @@ static char *make_request_msg(void)
 {
 	char *ret, *hn = para_hostname();
 	ret = make_message("%s1.0\nHost: %s\nUser-Agent: para_recv/%s\n\n\n",
-		HTTP_GET_MSG, hn, VERSION);
+		HTTP_GET_MSG, hn, PACKAGE_VERSION);
 	free(hn);
 	return ret;
 }
@@ -111,48 +111,48 @@ static void http_recv_post_select(struct sched *s, struct task *t)
 	struct private_http_recv_data *phd = rn->private_data;
 
 	t->ret = -E_HTTP_RECV_EOF;
-	if (rn->output_eof && *rn->output_eof) {
-		rn->eof = 1;
-		return;
-	}
+	if (rn->output_eof && *rn->output_eof)
+		goto out;
 	t->ret = 1;
-	if (!s->select_ret) /* we're not interested in timeouts */
-		return;
+	if (!s->select_ret)
+		goto out;
 	if  (phd->status == HTTP_CONNECTED) {
 		char *rq;
 		if (!FD_ISSET(phd->fd, &s->wfds))
-			return; /* nothing to do */
+			goto out;
 		rq = make_request_msg();
-		PARA_NOTICE_LOG("%s", "sending http request\n");
+		PARA_INFO_LOG("%s", "sending http request\n");
 		t->ret = send_va_buffer(phd->fd, "%s", rq);
 		free(rq);
 		if (t->ret > 0)
 			phd->status = HTTP_SENT_GET_REQUEST;
-		return;
+		goto out;
 	}
 	if (!FD_ISSET(phd->fd, &s->rfds))
-		return; /* nothing to do */
+		goto out;
 	if (phd->status == HTTP_SENT_GET_REQUEST) {
 		t->ret = recv_pattern(phd->fd, HTTP_OK_MSG, MAXLINE);
 		if (t->ret < 0)
-			return;
-		PARA_NOTICE_LOG("%s", "received ok msg, streaming\n");
+			goto out;
+		PARA_INFO_LOG("%s", "received ok msg, streaming\n");
+		t->ret = 1;
 		phd->status = HTTP_STREAMING;
-		return;
+		goto out;
 	}
 	t->ret = -E_OVERRUN;
-	/* already streaming */
 	if (rn->loaded >= BUFSIZE)
-		return;
+		goto out;
 	t->ret = recv_bin_buffer(phd->fd, rn->buf + rn->loaded,
 		BUFSIZE - rn->loaded);
 	if (t->ret <= 0) {
-		rn->eof = 1;
 		if (!t->ret)
 			t->ret = -E_HTTP_RECV_EOF;
-		return;
+		goto out;
 	}
 	rn->loaded += t->ret;
+out:
+	if (t->ret < 0)
+		rn->eof = 1;
 }
 
 static void http_recv_close(struct receiver_node *rn)
@@ -184,22 +184,25 @@ static int http_recv_open(struct receiver_node *rn)
 	rn->buf = para_calloc(BUFSIZE);
 	rn->private_data = para_calloc(sizeof(struct private_http_recv_data));
 	phd = rn->private_data;
-	PARA_NOTICE_LOG("phd = %p, rn = %p\n", phd, rn);
-	ret = -E_HOST_INFO;
-	if (!(he = get_host_info(conf->host_arg)))
+	ret = get_host_info(conf->host_arg, &he);
+	if (!ret < 0)
 		goto err_out;
 	/* get new socket */
-	ret = -E_SOCKET;
-	if ((phd->fd = get_socket()) < 0)
+	ret = get_socket();
+	if (ret < 0)
 		goto err_out;
+	phd->fd = ret;
 	/* init their_addr */
 	init_sockaddr(&their_addr, conf->port_arg, he);
 	/* connect */
 	PARA_NOTICE_LOG("connecting to %s:%d\n", conf->host_arg,
 		conf->port_arg);
 	ret = para_connect(phd->fd, &their_addr);
-	if (ret < 0)
+	if (ret < 0) {
+		close(phd->fd);
 		goto err_out;
+	}
+	mark_fd_nonblock(phd->fd);
 	phd->status = HTTP_CONNECTED;
 	return 1;
 err_out: