home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #27 / NN_1992_27.iso / spool / comp / os / linux / 17389 < prev    next >
Encoding:
Text File  |  1992-11-20  |  10.6 KB  |  404 lines
  1. Newsgroups: comp.os.linux
  2. Path: sparky!uunet!munnari.oz.au!metro!extro.ucc.su.OZ.AU!arumble
  3. From: arumble@extro.ucc.su.OZ.AU (Anthony Rumble)
  4. Subject: Re: Seyon, PATCH FOR SEYON1.0
  5. Message-ID: <arumble.722312451@extro.ucc.su.OZ.AU>
  6. Sender: news@ucc.su.OZ.AU
  7. Nntp-Posting-Host: extro.ucc.su.oz.au
  8. Organization: Sydney University Computing Service, Sydney, NSW, Australia
  9. References: <1992Nov20.161114.20044@cs.tulane.edu>
  10. Date: Sat, 21 Nov 1992 02:20:51 GMT
  11. Lines: 391
  12.  
  13. kranz@cs.tulane.edu (Barry Kranz) writes:
  14.  
  15. >Although I have been able to compile the X app Seyon, everytime I
  16. >execute it I get a "killed by signal 11" error, has anyone gotten
  17. >this to work?
  18.  
  19. >Any help would be greatly appreciated!
  20.  
  21. Yes.. While fixing the LOCKFILE stuff, I found a MAJOR
  22. bug/nasty.. He should be rapped over the knuckles
  23. for this one..
  24.  
  25. Basically, he did something like 
  26. lckf[40] = LOCKFILE;  (LOCKFILE being "/usr/spool/uucp/LCK.."
  27.  
  28. Which is illegal.
  29.  
  30. Here is a patch that fixes that bug and implementes CORRECT
  31. lock files...
  32.  
  33. -------- Cut here -------
  34.  
  35. diff +show-c-function +new-file seyon/Makefile seyon-1.0//Makefile
  36. *** seyon/Makefile    Sat Nov 21 04:15:21 1992
  37. --- seyon-1.0//Makefile    Mon Nov 16 04:44:29 1992
  38. *************** DEFS         = -DHELPFILE=$(HELPFILE)
  39. *** 43,49 ****
  40.   LIBS        = -lXaw -lXmu -lXt -lX11
  41.   OBJS        = Seyon.o SeTerm.o xcsubs.o SePort.o \
  42.                   xcscrpt.o SeMain.o SeWin.o SeSubs.o SeHelp.o SeSet.o \
  43. !               SeDial.o SeTrans.o SeMisc.o lockfile.o
  44.   .c.o:;    $(CC) $(CFLAGS) -c $<
  45.   
  46.   all:: seyon-cmd seyon
  47. --- 43,49 ----
  48.   LIBS        = -lXaw -lXmu -lXt -lX11
  49.   OBJS        = Seyon.o SeTerm.o xcsubs.o SePort.o \
  50.                   xcscrpt.o SeMain.o SeWin.o SeSubs.o SeHelp.o SeSet.o \
  51. !               SeDial.o SeTrans.o SeMisc.o
  52.   .c.o:;    $(CC) $(CFLAGS) -c $<
  53.   
  54.   all:: seyon-cmd seyon
  55. diff +show-c-function +new-file seyon/SePort.c seyon-1.0//SePort.c
  56. *** seyon/SePort.c    Sat Nov 21 14:27:18 1992
  57. --- seyon-1.0//SePort.c    Mon Nov 16 10:14:41 1992
  58. ***************
  59. *** 27,33 ****
  60.   
  61.   #include "seyon.h"
  62.   #include "SeDecl.h"
  63. - #include "lockfile.h"
  64.   
  65.   /*
  66.       The dial() routine uses these two defines.
  67. --- 27,32 ----
  68. *************** send_slowly(s)
  69. *** 427,449 ****
  70.       Simple, eh?
  71.   */
  72.   
  73.   
  74.   lock_tty()
  75.   {
  76. !     char lckf[40];
  77. !     int tmp;    
  78.       char *modemname;
  79.   
  80. !     strcpy(lckf, LOCKFILE);
  81.       modemname = strrchr(port, '/');
  82.       strcat(lckf, (modemname ? (modemname+1) : port));
  83. !     tmp = makelock(lckf);
  84.   
  85. !     if (tmp)
  86. !     return 0;
  87.   }
  88.   
  89.   unlock_tty()
  90.   {
  91. !     rmlocks();
  92.   }
  93. --- 426,500 ----
  94.       Simple, eh?
  95.   */
  96.   
  97. + char lckf[40] = LOCKFILE;
  98. + char ltmp[40] = LTMPFILE;
  99.   
  100.   lock_tty()
  101.   {
  102. !     int pid, lckpid, lfd;
  103. !     char pidstr[20];
  104. !     char lckpidstr[20];
  105.       char *modemname;
  106. +     extern int errno;
  107.   
  108. !     /*
  109. !      * Get our PID, and initialize the filename strings.
  110. !      */
  111. !     pid = getpid();
  112. !     sprintf(pidstr, "%d", pid);
  113.       modemname = strrchr(port, '/');
  114.       strcat(lckf, (modemname ? (modemname+1) : port));
  115. !     strcat(ltmp, pidstr);
  116.   
  117. !     /*
  118. !      * Create the LTMP.<pid> file and scribble our PID in it.
  119. !      */
  120. !     unlink(ltmp);
  121. !     if ((lfd = creat(ltmp, 0444)) == -1) {
  122. !     fprintf(stderr, "Can't creat(2) LTMP file in %s\r\n", LTMPFILE);
  123. !     return -1;
  124. !     }
  125. !     sprintf(pidstr, "%10d\n", pid);
  126. !     write(lfd, pidstr, 11);
  127. !     close(lfd);
  128. ! 
  129. !     /*
  130. !      * Attempt to link directly - if it works, we're done.
  131. !      */
  132. !   relink:
  133. !     if (link(ltmp, lckf) == 0) {
  134. !     unlink(ltmp);
  135. !         return 0;
  136. !     }
  137. ! 
  138. !     /*
  139. !      * Oh, bother, there's a LCK..* file already; we must
  140. !      * now expend effort to learn if it's stale or not.
  141. !      */
  142. !     if ((lfd = open(lckf, O_RDONLY)) != -1) {
  143. !     if (read(lfd, lckpidstr, 11) == 11) {
  144. !         lckpid = atol(lckpidstr);
  145. !         if (kill(lckpid, 0) == 0) {
  146. !         fprintf(stderr, "%s locked by process %d.\r\n", port, lckpid);
  147. !         unlink(ltmp);
  148. !         return -1;
  149. !         }
  150. !     }
  151. !     }
  152. ! 
  153. !     /*
  154. !      * The LCK..* file was stale.  Remove and retry.
  155. !      */
  156. !     if (unlink(lckf)) {
  157. !     fprintf(stderr, "Can't unlink(2) stale LCK?\r\n");
  158. !     unlink(ltmp);
  159. !     return -1;
  160. !     }
  161. !     goto relink;
  162. !     /*NOTREACHED*/
  163.   }
  164.   
  165.   unlock_tty()
  166.   {
  167. !     unlink(lckf);
  168.   }
  169. diff +show-c-function +new-file seyon/lockfile.c seyon-1.0//lockfile.c
  170. *** seyon/lockfile.c    Sat Nov 21 04:10:59 1992
  171. --- seyon-1.0//lockfile.c
  172. ***************
  173. *** 1,182 ****
  174. - /*************************************************************************
  175. - General UUCP Lock file Code (For NIXMAIL)
  176. - --------------------------------------------------------------------------
  177. - Copyright (c) 1992 Anthony Rumble
  178. - --------------------------------------------------------------------------
  179. - RCS Info
  180. - 
  181. - $Header: /home/smilie/emsi/RCS/lockfile.c,v 1.7 1992/10/18 07:57:51 smilie Exp $
  182. - 
  183. - $Log: lockfile.c,v $
  184. -  * Revision 1.7  1992/10/18  07:57:51  smilie
  185. -  * added rmlock
  186. -  *
  187. -  * Revision 1.6  1992/10/16  08:58:01  smilie
  188. -  * updated for nixmail
  189. -  *
  190. -  * Revision 1.5  1992/10/10  05:58:36  smilie
  191. -  * took out atexit
  192. -  *
  193. -  * Revision 1.4  1992/10/09  13:36:46  smilie
  194. -  * fixed some warnings
  195. -  *
  196. -  * Revision 1.3  1992/10/09  13:05:45  smilie
  197. -  * fixed a bug with the kill return
  198. -  *
  199. -  * Revision 1.2  1992/10/09  12:56:02  smilie
  200. -  * fixed an debug message up
  201. -  *
  202. -  * Revision 1.1  1992/10/09  09:57:26  smilie
  203. -  * Initial revision
  204. -  *
  205. - 
  206. - *************************************************************************/
  207. - 
  208. - /* Feature test switches */
  209. - #define _POSIX_SOURCE 1
  210. - #define _LOCKFILE_C
  211. - 
  212. - /* System Headers */
  213. - #include <stdio.h>
  214. - #include <stdlib.h>
  215. - #include <sys/types.h>
  216. - #include <unistd.h>
  217. - #include <signal.h>
  218. - #include <string.h>
  219. - 
  220. - /* Local Headers */
  221. - #include "lockfile.h"
  222. - 
  223. - /* Macros */
  224. - 
  225. - /* File scope variables */
  226. - 
  227. - static char lockfile_rcsid[] = "$Id: lockfile.c,v 1.7 1992/10/18 07:57:51 smilie Exp $";
  228. - #define RCSID lockfile_rcsid
  229. - 
  230. - /* External variables */
  231. - 
  232. - char lock[256];        /* Store the current lockfile */
  233. - 
  234. - /* External Functions */
  235. - 
  236. - /* Structures and unions */
  237. - 
  238. - /* Functions */
  239. - 
  240. - /************************************************************************
  241. -                 RMLOCKS
  242. - -------------------------------------------------------------------------
  243. -              Remove the current UUCP locks
  244. - *************************************************************************/
  245. - void rmlocks(void)
  246. - {
  247. - (void)unlink(lock);
  248. - }
  249. - /************************************************************************
  250. -                 RMLOCK
  251. - -------------------------------------------------------------------------
  252. - *************************************************************************/
  253. - void rmlock(char *s)
  254. - {
  255. - (void)unlink(s);
  256. - }
  257. - /************************************************************************
  258. -                 READLOCK
  259. - -------------------------------------------------------------------------
  260. - Read the PID from a lockfile, return -1 if there is no
  261. - valid lockfile
  262. - *************************************************************************/
  263. - pid_t readlock(char *name)
  264. - {
  265. - pid_t pid;
  266. - FILE *fh;
  267. - /**/
  268. - 
  269. - if ((fh = fopen(name, "rb")) == NULL)            /* Open the lockfile */
  270. -     {
  271. -     #ifdef DEBUG
  272. -     fprintf(stderr, "readlock: %s : %s\n", name, strerror(errno));
  273. -     #endif
  274. -     return -1;                    /* No lockfile exists */
  275. -     }
  276. - 
  277. - (void)fread(&pid, sizeof(pid), 1, fh);            /* Read in PID */
  278. - (void)fclose(fh);
  279. - return pid;
  280. - }
  281. - /************************************************************************
  282. -                 CHECKLOCK
  283. - -------------------------------------------------------------------------
  284. - Check the current lock, return TRUE if a valid lock exists
  285. - return FALSE if there is no valid lock
  286. - 
  287. - Pass the full path and filename of the lockfile to name
  288. - *************************************************************************/
  289. - int checklock(char *name)
  290. - {
  291. - pid_t pid;                /* PID Variable */
  292. - /**/
  293. - 
  294. - pid = readlock(name);            /* Read in PID */
  295. - if (pid == -1)                
  296. -     {
  297. -     #ifdef DEBUG
  298. -     (void)fprintf(stderr, "checklock: no valid lockfile exists\n");
  299. -     #endif
  300. -     return FALSE;
  301. -     }
  302. - 
  303. - if (kill(pid, 0) == -1)        /* See if the PID actually exists */
  304. -     {
  305. -     if (errno == ESRCH)
  306. -         {
  307. -         #ifdef DEBUG
  308. -         (void)fprintf(stderr, "checklock: PID '%u' does not exist, removing lockfile\n", pid);
  309. -         #endif
  310. -         (void)unlink(name);    /* Remove lock file as there is no such PID */
  311. -         return FALSE;
  312. -         }
  313. -     (void)fprintf(stderr, "checklock: Unknown KILL return '%u'\n", errno);
  314. -     return FALSE;
  315. -     }
  316. -     
  317. - return TRUE;                /* A valid lockfile was found */
  318. - }
  319. - /************************************************************************
  320. -                 MAKELOCK
  321. - -------------------------------------------------------------------------
  322. -      Will creat the lockfile 'name' if there is not a valid lockfile
  323. -                 under this name.
  324. - *************************************************************************/
  325. - int makelock(char *name)
  326. - {
  327. - FILE *fh;
  328. - pid_t pid;
  329. - /**/
  330. - 
  331. - if (checklock(name))
  332. -     {
  333. -     (void)fprintf(stderr, "makelock: Error, Valid lock file exists\n");
  334. -     return FALSE;
  335. -     }
  336. - 
  337. - pid = getpid();                        /* Get current PID */
  338. - 
  339. - if ((fh = fopen(name, "wb")) == NULL)
  340. -     {
  341. -     (void)fprintf(stderr, "Error creating lockfile '%s'\n", name);
  342. -     return FALSE;
  343. -     }
  344. - 
  345. - (void)fwrite(&pid, sizeof(pid), 1, fh);            /* Write PID */
  346. - (void)fclose(fh);
  347. - #ifdef DEBUG
  348. - (void)fprintf(stderr, "makelock: lock made\n");
  349. - #endif
  350. - (void)strcpy(lock, name);                /* set global lock file */
  351. - return TRUE;
  352. - }
  353. - 
  354. - 
  355. - 
  356. --- 0 ----
  357. diff +show-c-function +new-file seyon/lockfile.h seyon-1.0//lockfile.h
  358. *** seyon/lockfile.h    Sat Nov 21 04:10:03 1992
  359. --- seyon-1.0//lockfile.h
  360. ***************
  361. *** 1,36 ****
  362. - /*************************************************************************
  363. - GENERAL Lockfile header file
  364. - --------------------------------------------------------------------------
  365. - Copyright (c) 1992 Anthony Rumble
  366. - --------------------------------------------------------------------------
  367. - RCS Info
  368. - 
  369. - $Header: /home/smilie/emsi/RCS/lockfile.h,v 1.1 1992/10/09 13:19:00 smilie Exp $
  370. - 
  371. - $Log: lockfile.h,v $
  372. -  * Revision 1.1  1992/10/09  13:19:00  smilie
  373. -  * Initial revision
  374. -  *
  375. - 
  376. - $Id: lockfile.h,v 1.1 1992/10/09 13:19:00 smilie Exp $
  377. - 
  378. - *************************************************************************/
  379. - #ifndef _LOCKFILE_H
  380. - #define _LOCKFILE_H
  381. - 
  382. - #include <sys/types.h>
  383. - 
  384. - extern char lock[256];            /* lock file name */
  385. - 
  386. - extern void rmlocks(void);        /* remove lock file */
  387. - extern pid_t readlock(char *);        /* read a lock file */
  388. - extern int checklock(char *);        /* check a lock file */
  389. - extern int makelock(char *);        /* make a lock file */
  390. - 
  391. - #ifndef TRUE
  392. - #define TRUE 1
  393. - #define FALSE 0
  394. - #endif
  395. - 
  396. - #endif
  397. - 
  398. --- 0 ----
  399. Common subdirectories: seyon/xcomm_docs and seyon-1.0//xcomm_docs
  400. -- 
  401. Anthony Rumble
  402. aka SmilieZ
  403. "Anything is possible.. If there is enuf money in it"
  404.