bugzilla-daemon at mindrot.org
2002-Mar-07 20:38 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mouring at eviladmin.org 2002-03-08 07:38 ------- I just went through someone with this problem. And HAVE_BOGUS_SYS_QUEUE_H worked for them. However you must have BOTH HAVE_SYS_QUEUE_H and HAVE_BOGUS_SYS_QUEUE_H set. As for INADDR_LOOPBACK. I'd like to know where on BSD/OS that is defined so we can check and include it correctly. Otherwise the correct fix (which I am verifying with the SCO 3 fokes) is to change it to #define INADDR_LOOPBACK (0x7f000001L) ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-08 00:38 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From spike at indra.com 2002-03-08 11:38 -------> I just went through someone with this problem. And HAVE_BOGUS_SYS_QUEUE_H > worked for them. However you must have BOTH HAVE_SYS_QUEUE_H and > HAVE_BOGUS_SYS_QUEUE_H set.I tried this, and had no luck. As a test I even removed the #ifdefs in ssh-agent.c to force fake-queue.h to be included. If I'm reading gcc -E right, and it's been a long time since I have, the problem lies in <sys/queue.h> getting included before fake-queue.h. The include path looks like: includes.h -> config.h -> defines.h -> <netinet/in.h> -> <netinet6/in6.h> -> <sys/queue.h> Then <sys/queue.h> defines _SYS_QUEUE_H_ so that when fake-queue.h, it's all ifdef'ed out (and of course it would explode if both were included). I pasted the define for TAILQ_FOREACH into ssh-agent.c and it compiles fine. So it's probably just a case of finding a more graceful way of doing that.> As for INADDR_LOOPBACK. I'd like to know where on BSD/OS that is defined > so we can check and include it correctly.It's defined in <rpc/types.h>: #ifndef INADDR_LOOPBACK #define INADDR_LOOPBACK (u_long)0x7F000001 #endif I included that in defines.h and it compiled as well. Having done that I installed 3.1p1. The server works fine, but the clients all failed with: (rand child) setuid: Operation not permitted ssh-rand-helper child produced insufficient data Removing the setuid bit from "ssh" solved this problem. Digging a little deeper it looks like original_uid and original_euid are never initialized. ->Spike ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-14 18:17 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mdev at idg.nl 2002-03-15 05:17 ------- Created an attachment (id=44) sftp-int.c: cmds[] to sftp_cmds[] ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-14 18:19 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mdev at idg.nl 2002-03-15 05:19 ------- On a 4.1 system I had the same errors and an additional one. The fake-queue, was fixable by: CFLAGS=-DHAVE_BOGUS_SYS_QUEUE_H=1 The additional error is in sftp-int.c. The build fails on: gcc -o sftp sftp.o sftp-client.o sftp-common.o sftp-int.o sftp-glob.o -L. -Lopenbsd-compat/ -L/weblib/local/lib -L/usr/lib -Lyes -lssh -l openbsd-compat -lutil -lz -lcrypto /usr/lib/libc.a(prandom.o)(.data+0x0): multiple definition of `cmds' sftp-int.o(.rodata+0x48): first defined here ld: Warning: size of symbol `cmds' changed from 256 to 40 in prandom.o make: *** [sftp] Error 1 I renamed cmds to sftp_cmds and all is well. See uploaded patch. ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-14 19:53 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mouring at eviladmin.org 2002-03-15 06:52 ------- Why are we renaming things without any reasoning behind it? ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-15 07:22 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mdev at idg.nl 2002-03-15 18:22 ------- The reasoning is, that this bug is similar to: http://www.openldap.org/lists/openldap-bugs/200005/msg00032.html And that prandom.o is in libc. Another reason is, that 'cmds' is likely to conflict unless you protect it, since it's quite a common name. I don't know the reason for using sftp's own structure, instead of using the libc provided structure, so if it can be modified to use libc's. The only reference to cmds I found in /usr/include was: /usr/include/arpa/telnet.h Which also uses it's own name: #ifdef TELCMDS char *telcmds[] = { "EOF", "SUSP", "ABORT", "EOR", "SE", "NOP", "DMARK", "BRK", "IP", "AO", "AYT", "EC", "EL", "GA", "SB", "WILL", "WONT", "DO", "DONT", "IAC", 0, }; #else extern char *telcmds[]; #endif So it seemed the logical thing to do. ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-17 20:22 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From markus at openbsd.org 2002-03-18 07:22 ------- why not have the array static? ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-18 01:54 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mdev at idg.nl 2002-03-18 12:54 ------- Created an attachment (id=47) Workaround for fake-queue ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-18 02:03 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 ------- Additional Comments From mdev at idg.nl 2002-03-18 13:03 ------- The patch for fake-queue.h works, but just looks ugly. Since <sys/param.h> is the one <sys/queue.h>, and the full path becomes: "includes.h" -> "config.h" -> "defines.h" -> <sys/param.h> I wasn't able to come up with a better solution. The #undef's are there to get rid of the 'redefinition' warnings. That it works on 4.1 is probably, because for some reason somewhere, KERNEL is defined on 4.2 before including <sys/param.h> while it isn't on 4.1. As for the static cmds[], that seemed to do the trick quite well. ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-18 12:05 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 mdev at idg.nl changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tansympp at singnet.com.sg ------- Additional Comments From mdev at idg.nl 2002-03-18 23:05 ------- *** Bug 155 has been marked as a duplicate of this bug. *** ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Mar-19 20:21 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 markus at openbsd.org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |deven at netexplorer.org ------- Additional Comments From markus at openbsd.org 2002-03-20 07:21 ------- *** Bug 174 has been marked as a duplicate of this bug. *** ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
bugzilla-daemon at mindrot.org
2002-Apr-05 20:32 UTC
[Bug 146] OpenSSH 3.1p1 will not build on BSD/OS 4.2/4.1/4.01
http://bugzilla.mindrot.org/show_bug.cgi?id=146 mouring at eviladmin.org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- Additional Comments From mouring at eviladmin.org 2002-04-06 06:32 ------- I just commited a patch that does the following: - Removes all attempt to detect/use system <sys/queue.h> - #undef all macros in fake-queue.h - changes the #ifdef/#define/#endif wrapper from _SYS_QUEUE to _FAKE_QUEUE - Uses our fake-queue.h always It should be out on the CVS mirrors within a few hours. If this does not resolve the issue please reopen before 3.2 release. - Ben ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.