Opened 10 years ago
Closed 6 years ago
#44279 closed defect (fixed)
screen 4.2.1_0 builds okay, but crashes with "Abort trap"
Reported by: | alexander.coers@… | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.1 |
Keywords: | Cc: | ||
Port: | screen |
Description
On my OS X 10.5.8 PPC G4 with Xcode 3.1.4 I can build screen 4.2.1_0 without errors. Starting screen with -v will produce the correct version output. Any other option (or none) will just show an "Abort trap" crash. Going back to the earlier version 4.0.3_3+vertical_split using revision 95934 solves the issue.
Change History (3)
comment:1 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
comment:2 Changed 10 years ago by irgendetwas@…
This appears to be because the realpath function under osx 10.5.8 does not handle the second argument being NULL as per spec.
in tty.c adding a buffer for realpath to work in and removing the free at the end allows screen to work properly under osx 10.5.8.
now if someone who knows how to make the patches and fix the port files wants to correct this, it would be much appreciated :-)
int CheckTtyname (tty) char *tty; { struct stat st; char * real; char realbuf[PATH_MAX]; //-------- buffer added int rc; real = realpath(tty, realbuf); //----------- NULL -> realbuf if (!real) return -1; if (lstat(real, &st) || !S_ISCHR(st.st_mode) || (st.st_nlink > 1 && strncmp(real, "/dev/", 5))) rc = -1; else rc = 0; //free(real); //-------------- commented out return rc; }
comment:3 Changed 6 years ago by dgilman (David Gilman)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
MacPorts currently has no maintainer for the screen port, so you should report this to screen's developers. However, note that Mac OS X 10.5.8 is very old and the developers may not be able to solve this issue.