Article View: gnu.bash.bug
Article #370bash bug report
From: Brian Fox
Date: Fri, 21 Jul 1989 22:30
Date: Fri, 21 Jul 1989 22:30
145 lines
5237 bytes
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!bfoxReferences:
<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