QubesOS/qubes-core-admin-linux
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Revert "qrexec: fix deadlock in qrexec-client"

Browse Source 
This reverts commit 79abec9038.

The problem will not be applicable in new protocol, where vchan
connection is directly between VMs, so there is no longer two connected
qrexec-clients - always one end of data flow in qrexec-client is vchan,
which provide information about amount of data to read or buffer
space to write (lack of the later in case of pipes was a cause of the
original problem).
This commit is contained in:
Marek Marczykowski-Górecki 2014-09-10 00:12:18 +02:00
parent 9a1c071f40
commit 6e47f12118
2 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
qrexec/Makefile
View File

@ -7,6 +7,6 @@ all: qrexec-daemon qrexec-client
qrexec-daemon: qrexec-daemon.o qrexec-daemon: qrexec-daemon.o
$(CC) -pie -g -o qrexec-daemon qrexec-daemon.o $(LIBS) $(CC) -pie -g -o qrexec-daemon qrexec-daemon.o $(LIBS)
qrexec-client: qrexec-client.o qrexec-client: qrexec-client.o
$(CC) -pie -g -o qrexec-client qrexec-client.o $(LIBS) -lpthread $(CC) -pie -g -o qrexec-client qrexec-client.o $(LIBS)
clean: clean:
rm -f *.o *~ qrexec-daemon qrexec-client rm -f *.o *~ qrexec-daemon qrexec-client

36
qrexec/qrexec-client.c
View File

@ -27,7 +27,6 @@
#include <unistd.h> #include <unistd.h>
#include <sys/wait.h> #include <sys/wait.h>
#include <errno.h> #include <errno.h>
#include <pthread.h>
#include "qrexec.h" #include "qrexec.h"
#include "libqrexec-utils.h" #include "libqrexec-utils.h"
@ -226,25 +225,30 @@ void handle_daemon_only_until_writable(int s)
} while (!FD_ISSET(s, &wrset)); } while (!FD_ISSET(s, &wrset));
} }
void *input_process_loop(void *arg) {
int s = *(int*)arg;
while (local_stdout_fd != -1)
handle_input(s);
return NULL;
}
void select_loop(int s) void select_loop(int s)
{ {
pthread_t input_thread; fd_set select_set;
if (pthread_create(&input_thread, NULL, input_process_loop, &s) != 0) { int max;
perror("pthread_create");
do_exit(1);
}
for (;;) { for (;;) {
handle_daemon_data(s); handle_daemon_only_until_writable(s);
FD_ZERO(&select_set);
FD_SET(s, &select_set);
max = s;
if (local_stdout_fd != -1) {
FD_SET(local_stdout_fd, &select_set);
if (s < local_stdout_fd)
max = local_stdout_fd;
}
if (select(max + 1, &select_set, NULL, NULL, NULL) < 0) {
perror("select");
do_exit(1);
}
if (FD_ISSET(s, &select_set))
handle_daemon_data(s);
if (local_stdout_fd != -1
&& FD_ISSET(local_stdout_fd, &select_set))
handle_input(s);
} }
pthread_join(input_thread, NULL);
} }
void usage(const char *name) void usage(const char *name)