🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: gnu.bash.bug
Article #370

bash bug report

#370
From: Brian Fox
Date: Fri, 21 Jul 1989 22:30
145 lines
5237 bytes
   Date: Fri, 21 Jul 89 15:14:08 PDT
   From: emil%chem@ucsd.edu (Emil M. Scoffone)
   To: bug-bash@ai.mit.edu

   GNU program:		Bash 1.02
   Machine:		Compaq Deskpro 386/16
   Operating System:	System V/386 Release 3.2
			   (vendor: Bell Technologies)
   Compiler:		GNU C, version 1.35, with GNU as and ld.

   ****************************************

   These are bugs I found while installing bash:

   ----------------------------------------
   The shell does incorrectly reports exit codes from a killed job.
   This is because a couple of masks were defined incorrectly in
   `jobs.h'. I also parenthesized two expressions in 'nojobs.h'.
   ----------------------------------------

Thank you for your clear bug report.

I recently received a fix for the 386 problems.  It has to do with the
byte order defined in the "wait" structure.  Please try using this
jobs.h file, and tell me if it works.

----------------------------------------
File: /usr/gnu/src/bash/jobs.h
----------------------------------------
/* jobs.h -- structures and stuff used by the jobs.c file. */

#if !defined (SYSV) || defined (UNIXPC)
#include <sys/wait.h>
#else

#if defined (i386) || defined (ATT3B) || defined (ATT386)

union wait {
  struct {
    unsigned char low;
    unsigned char high;
  }
  bytes;
  short word;
};

#define WSTOPPED 0177
#define w_status word
#define w_termsig bytes.low & ~0x7f
#define w_stopsig bytes.high
#define w_retcode bytes.high
#define w_coredump w_termsig & 0x7f
#define WIFEXITED(wstat) ((wstat).bytes.low == 0)
#define WIFSTOPPED(wstat) ((wstat).bytes.low == 0177)
#define WIFTERMINATED(wstat) ((wstat).bytes.high == 0)

#else  /* i386 || ATT3B || ATT386 */

#ifdef HPUX
union wait
  {
    int	w_status;		/* used in syscall */

    /* Terminated process status. */
    struct
      {
	unsigned short
	  w_Fill1    : 16,	/* high 16 bits unused */
	  w_Retcode  : 8,	/* exit code if w_termsig==0 */
	  w_Coredump : 1,	/* core dump indicator */
	  w_Termsig  : 7;	/* termination signal */
      } w_T;

    /* Stopped process status.  Returned
       only for traced children unless requested
       with the WUNTRACED option bit. */
    struct
      {
	unsigned short
	  w_Fill2   : 16,	/* high 16 bits unused */
	  w_Stopsig : 8,	/* signal that stopped us */
	  w_Stopval : 8;	/* == W_STOPPED if stopped */
      } w_S;
  };

#define	w_termsig w_T.w_Termsig
#define w_coredump w_T.w_Coredump
#define w_retcode w_T.w_Retcode
#define w_stopval w_S.w_Stopval
#define w_stopsig w_S.w_Stopsig

#define	WSTOPPED 0177
#define WIFSTOPPED(x) (((x) . w_stopval) == WSTOPPED)
#define WIFEXITED(x) ((! (WIFSTOPPED (x))) && (((x) . w_termsig) == 0))
#define WIFSIGNALED(x) ((! (WIFSTOPPED (x))) && (((x) . w_termsig) != 0))

#endif  /* HPUX */
#endif  /* i386 || ATT3B || ATT386 */
#endif  /* !defined (SYSV) || defined (UNIXPC) */

/* I looked it up.  For pretty_print_job ().  The real answer is 24. */
#define LONGEST_SIGNAL_DESC 20

/* We keep an array of jobs.  Each entry in the array is a linked list
   of processes that are piped together.  The first process encountered is
   the group leader. */

/* Each child of the shell is remembered in a STRUCT PROCESS.  A chain of
   such structures is a pipeline.  The chain is circular. */
typedef struct process {
  struct process *next;	/* Next process in the pipeline.  A circular chain. */
  int pid;		/* Process ID. */
  union wait status;	/* The status of this command as returned by wait. */
  int running;		/* Non-zero if this process is running. */
  char *command;	/* The particular program that is running. */
} PROCESS;

/* A description of a pipeline's state. */
typedef enum { JRUNNING, JSTOPPED, JDEAD, JMIXED } JOB_STATE;
#define JOBSTATE(job) (jobs[(job)]->state)

typedef struct job {
  char *wd;		/* The working directory at time of invocation. */
  PROCESS *pipe;	/* The pipeline of processes that make up this job. */
  int pgrp;		/* The process ID of the process group (necessary). */
  int foreground;	/* Non-zero if this is running in the foreground. */
  int notified;		/* Non-zero if already notified about job state. */
  JOB_STATE state;	/* The state that this job is in. */
#ifdef JOB_CONTROL
  COMMAND *deferred;	/* Commands that will execute when this job is done. */
#endif
} JOB;

#define NO_JOB -1	/* An impossible job array index. */
#define DUP_JOB -2	/* A possible return value for get_job_spec (). */

/* Stuff from the jobs.c file. */
extern int last_made_pid;
extern int current_job, previous_job;
extern int asynchronous_notification;
extern JOB **jobs;
extern int job_slots;

----------------------------------------
Technical Details
Message-ID: <8907212230.AA17486@aurel.caltech.edu>
Path: arc1-ca.pugleaf.net!utzoo!utgpu!jarvis.csri.toronto.edu!mailrus!tut.cis.ohio-state.edu!AUREL.CALTECH.EDU!bfox
References: <8907212214.AA19076@sdchemv1.chem.ucsd.edu>

Show Full Headers
Complete Article Headers
Path: arc1-ca.pugleaf.net!utzoo!utgpu!jarvis.csri.toronto.edu!mailrus!tut.cis.ohio-state.edu!AUREL.CALTECH.EDU!bfox
From: bfox@AUREL.CALTECH.EDU (Brian Fox)
Newsgroups: gnu.bash.bug
Subject: bash bug report
Message-ID: <8907212230.AA17486@aurel.caltech.edu>
Date: 21 Jul 89 22:30:02 GMT
References: <8907212214.AA19076@sdchemv1.chem.ucsd.edu>
Sender: daemon@tut.cis.ohio-state.edu
Reply-To: bfox@aurel.caltech.edu
Distribution: gnu
Organization: GNUs Not Usenet
Lines: 145