Merge remaining 4 PRs: memory safety, refactoring, test coverage, integration cleanup #93
@@ -138,24 +138,19 @@ Client* client_connect_ssh(const char* destination, int port) {
|
|||||||
ssh_argv[ac++] = "-o";
|
ssh_argv[ac++] = "-o";
|
||||||
ssh_argv[ac++] = "ControlPath=~/.cache/fastsync-%r@%h:%p";
|
ssh_argv[ac++] = "ControlPath=~/.cache/fastsync-%r@%h:%p";
|
||||||
if (port > 0 && port != 22) {
|
if (port > 0 && port != 22) {
|
||||||
if ((size_t)ac + 2 >= ssh_argv_max) {
|
if ((size_t)ac + 2 >= ssh_argv_max)
|
||||||
free(ssh_argv);
|
|
||||||
_exit(1);
|
_exit(1);
|
||||||
}
|
|
||||||
ssh_argv[ac++] = "-p";
|
ssh_argv[ac++] = "-p";
|
||||||
snprintf(port_str, sizeof(port_str), "%d", port);
|
snprintf(port_str, sizeof(port_str), "%d", port);
|
||||||
ssh_argv[ac++] = port_str;
|
ssh_argv[ac++] = port_str;
|
||||||
}
|
}
|
||||||
if ((size_t)ac + 3 >= ssh_argv_max) {
|
if ((size_t)ac + 3 >= ssh_argv_max)
|
||||||
free(ssh_argv);
|
|
||||||
_exit(1);
|
_exit(1);
|
||||||
}
|
|
||||||
ssh_argv[ac++] = ssh_user;
|
ssh_argv[ac++] = ssh_user;
|
||||||
ssh_argv[ac++] = "fastsync-server";
|
ssh_argv[ac++] = "fastsync-server";
|
||||||
ssh_argv[ac++] = "--stdio";
|
ssh_argv[ac++] = "--stdio";
|
||||||
ssh_argv[ac] = NULL;
|
ssh_argv[ac] = NULL;
|
||||||
execvp("ssh", ssh_argv);
|
execvp("ssh", ssh_argv);
|
||||||
free(ssh_argv);
|
|
||||||
perror("exec of ssh failed");
|
perror("exec of ssh failed");
|
||||||
ssize_t wret = write(exec_pipe[1], "x", 1);
|
ssize_t wret = write(exec_pipe[1], "x", 1);
|
||||||
(void)wret;
|
(void)wret;
|
||||||
|
|||||||
@@ -179,7 +179,7 @@ static void test_receive_str_oversized() {
|
|||||||
size_t huge = MAX_STRING_SIZE + 1;
|
size_t huge = MAX_STRING_SIZE + 1;
|
||||||
EXPECT_TRUE(send_n_data(0, &huge, sizeof(size_t)));
|
EXPECT_TRUE(send_n_data(0, &huge, sizeof(size_t)));
|
||||||
|
|
||||||
char* received = receive_str(0);
|
const char* received = receive_str(0);
|
||||||
EXPECT_NULL(received);
|
EXPECT_NULL(received);
|
||||||
|
|
||||||
close(p[0]);
|
close(p[0]);
|
||||||
|
|||||||
Reference in New Issue
Block a user