From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-7472-mason-zsh=primenet.com.au@sunsite.auc.dk>
Received: (qmail 26658 invoked from network); 24 Aug 1999 12:16:38 -0000
Received: from sunsite.auc.dk (130.225.51.30)
  by ns1.primenet.com.au with SMTP; 24 Aug 1999 12:16:38 -0000
Received: (qmail 22783 invoked by alias); 24 Aug 1999 12:16:32 -0000
Mailing-List: contact zsh-workers-help@sunsite.auc.dk; run by ezmlm
Precedence: bulk
X-No-Archive: yes
X-Seq: 7472
Received: (qmail 22776 invoked from network); 24 Aug 1999 12:16:24 -0000
Date: Tue, 24 Aug 1999 14:16:18 +0200 (MET DST)
Message-Id: <199908241216.OAA09744@beta.informatik.hu-berlin.de>
From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@sunsite.auc.dk
Subject: PATCH: fix for SEGV


Try this:

  ...) ./zsh -f
  % setopt autocd; hash wc=/bin/wc; echo|wc

It'll give you a SEGV in `isreallycom()', trying to look at
`cn->u.name' which isn't set because for hashed commands `cn->u.cmd'
is used.

Bye
 Sven

--- os/exec.c	Fri Aug 20 15:18:07 1999
+++ Src/exec.c	Tue Aug 24 14:12:19 1999
@@ -615,12 +615,16 @@
 int
 isreallycom(Cmdnam cn)
 {
-    char fullnam[MAXCMDLEN];
+    if (cn->flags & HASHED)
+	return 1;
+    else {
+	char fullnam[MAXCMDLEN];
 
-    strcpy(fullnam, cn->u.name ? *(cn->u.name) : "");
-    strcat(fullnam, "/");
-    strcat(fullnam, cn->nam);
-    return iscom(fullnam);
+	strcpy(fullnam, cn->u.name ? *(cn->u.name) : "");
+	strcat(fullnam, "/");
+	strcat(fullnam, cn->nam);
+	return iscom(fullnam);
+    }
 }
 
 /**/

--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de