Displaying 8 results from an estimated 8 matches for "controlsocket".
2012 Apr 27
2
[PATCH] mux: fix memory leak of control path if bind() fails
...195,7 @@ muxserver_listen(void)
close(muxserver_sock);
muxserver_sock = -1;
}
+ xfree(orig_control_path);
xfree(options.control_path);
options.control_path = NULL;
options.control_master = SSHCTL_MASTER_NO;
@@ -1216,7 +1217,6 @@ muxserver_listen(void)
}
error("ControlSocket %s already exists, disabling multiplexing",
orig_control_path);
- xfree(orig_control_path);
unlink(options.control_path);
goto disable_mux_master;
}
--
1.7.9.rc0.542.g07ca1
2007 Jul 05
14
[Bug 1329] New: stale control sockets prevent connection.
http://bugzilla.mindrot.org/show_bug.cgi?id=1329
Summary: stale control sockets prevent connection.
Product: Portable OpenSSH
Version: 4.6p1
Platform: Other
OS/Version: Linux
Status: NEW
Severity: normal
Priority: P2
Component: ssh
AssignedTo: bitbucket at mindrot.org
ReportedBy: dwmw2 at
2008 Dec 04
0
[patch] muxserver_listen() doesn't restore umask when bind fails
Cheers,
- Salva
$ diff -c mux.c~ mux.c
*** mux.c~ 2008-06-14 01:01:54.000000000 +0200
--- mux.c 2008-12-04 12:24:45.000000000 +0100
***************
*** 148,153 ****
--- 148,154 ----
error("ControlSocket %s already exists, "
"disabling multiplexing", options.control_path);
close(muxserver_sock);
+ umask(old_umask);
muxserver_sock = -1;
xfree(options.control_path);
options.control_path = NULL;
2007 Jun 22
3
[Bug 1208] ssh fails to remove control socket when ssh is abnormally terminated
http://bugzilla.mindrot.org/show_bug.cgi?id=1208
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
AssignedTo|bitbucket at mindrot.org |djm at mindrot.org
--- Comment #17 from Damien
2007 Aug 03
1
race condition with ControlMaster=auto
...socket");
memset(&addr, '\0', sizeof(addr));
addr.sun_family = AF_UNIX;
@@ -1073,11 +1074,9 @@
old_umask = umask(0177);
if (bind(control_fd, (struct sockaddr *)&addr, addr_len) == -1) {
control_fd = -1;
- if (errno == EINVAL || errno == EADDRINUSE)
- fatal("ControlSocket %s already exists",
- options.control_path);
- else
+ if (errno != EINVAL && errno != EADDRINUSE)
fatal("%s bind(): %s", __func__, strerror(errno));
+ return 0;
}
umask(old_umask);
@@ -1085,6 +1084,9 @@
fatal("%s listen(): %s", __func__, strerr...
2007 Apr 19
1
Is it possible to add %u for local user to ControlPath's argument in the portable version of OpenSSH?
...localmachine->remoteaccount at remotemachine:22=
if now user2 on localmachine tries the same, he gets an error message
> user2 at localmachine ~ % ssh remoteaccount at remotemachine
> Control socket connect(/tmp/localmachine->remoteaccount at remotemachine:22): Permission denied
> ControlSocket /tmp/localmachine->remoteaccount at remotemachine:22 already exists
because user1 owns a connection sharing socket of that name already.
If one could use %u in ControlPath's argument, sockets would look like
> localmachine ~ # ls -l /tmp
> srw------- 1 user1 user1 0 19. Apr 14:06 use...
2007 Jul 21
2
tincctl patches
(Second try to send this. I wonder if the first one gotten eaten by a
spam filter; I'll link to patches instead of attaching them.)
Here are the tincctl patches I've been working on. They apply to
http://www.tinc-vpn.org/svn/tinc/branches/1.1@1545. I intend to commit
them once the crypto stuff's fixed. Since they're basically done, I'm
emailing them now for review and in case
2010 Jan 14
1
ssh(1) multiplexing rewrite
...void
+mux_master_session_cleanup_cb(int cid, void *unused)
+{
+ Channel *cc, *c = channel_by_id(cid);
- old_umask = umask(0177);
- if (bind(muxserver_sock, (struct sockaddr *)&addr, addr.sun_len) == -1) {
- muxserver_sock = -1;
- if (errno == EINVAL || errno == EADDRINUSE) {
- error("ControlSocket %s already exists, "
- "disabling multiplexing", options.control_path);
- close(muxserver_sock);
- muxserver_sock = -1;
- xfree(options.control_path);
- options.control_path = NULL;
- options.control_master = SSHCTL_MASTER_NO;
- return;
- } else
- fatal("%s...