From c1bdffe99b6fa6fdb0f43e0dec85d8bbfcab1572 Mon Sep 17 00:00:00 2001
From: Eric Andersen <andersen@codepoet.org>
Date: Thu, 26 Apr 2001 15:56:47 +0000
Subject: Another nice cleanup from Larry.  This adds a new last_char_is()
 function and uses it to avoid possible buffer underruns whn strlen is zero,
 and avoid the possible space-hogging inline of strlen() in several cases. 
 -Erik

---
 tar.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'tar.c')

diff --git a/tar.c b/tar.c
index 48284c00a..716f4ac30 100644
--- a/tar.c
+++ b/tar.c
@@ -706,7 +706,7 @@ static int readTarFile(int tarFd, int extractFlag, int listFlag,
 			case REGTYPE0:
 				/* If the name ends in a '/' then assume it is
 				 * supposed to be a directory, and fall through */
-				if (header.name[strlen(header.name)-1] != '/') {
+				if (last_char_is(header.name,'/')) {
 					if (tarExtractRegularFile(&header, extractFlag, tostdoutFlag)==FALSE)
 						errorFlag=TRUE;
 					break;
-- 
cgit v1.2.3