diff -up xtrans-1.2.5.orig/Xtranslcl.c xtrans-1.2.5/Xtranslcl.c --- xtrans-1.2.5.orig/Xtranslcl.c 2009-10-19 10:06:47.000000000 +0200 +++ xtrans-1.2.5/Xtranslcl.c 2009-10-19 10:11:34.000000000 +0200 @@ -163,6 +163,7 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr if (strlen(sun_path) > sizeof(sunaddr->sun_path) - 1) { PRMSG(1, "FillAddrInfo: path too long\n", 0, 0, 0); + xfree((char *) sunaddr); return 0; } strcpy (sunaddr->sun_path, sun_path); @@ -190,6 +191,7 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr if (strlen(peer_sun_path) > sizeof(p_sunaddr->sun_path) - 1) { PRMSG(1, "FillAddrInfo: peer path too long\n", 0, 0, 0); + xfree((char *) p_sunaddr); return 0; } strcpy (p_sunaddr->sun_path, peer_sun_path); @@ -352,6 +354,7 @@ TRANS(PTSOpenClient)(XtransConnInfo cipt if ((fd = open(DEV_PTMX, O_RDWR)) < 0) { PRMSG(1,"PTSOpenClient: failed to open %s\n", DEV_PTMX, 0,0); + close(server); return(-1); } @@ -1126,6 +1129,10 @@ TRANS(SCOOpenServer)(XtransConnInfo cipt if ((fds = open(DEV_SPX, O_RDWR)) < 0 || (fdr = open(DEV_SPX, O_RDWR)) < 0 ) { PRMSG(1,"SCOOpenServer: failed to open %s\n", DEV_SPX, 0,0 ); + if (fds >= 0) + close(fds); + if (fdr >= 0) + close(fdr); return -1; } @@ -1165,7 +1172,7 @@ TRANS(SCOOpenServer)(XtransConnInfo cipt } fdr = open (serverR_path, O_RDWR | O_NDELAY); - if (fds < 0) { + if (fdr < 0) { PRMSG(1,"SCOOpenServer: failed to open %s\n", serverR_path, 0, 0); close (fds); return -1;