From 05d1bf1f462fc956ecdc5087c6e9f5241b1430da Mon Sep 17 00:00:00 2001 From: Perry Ruiter Date: Thu, 29 May 2014 19:24:24 -0700 Subject: [PATCH] afs: Delete unneeded duplicate code Delete a memory release and a goto since natural code flow will result in exactly the same thing. Reviewed-on: http://gerrit.openafs.org/11179 Tested-by: BuildBot Reviewed-by: Chas Williams - CONTRACTOR Reviewed-by: Benjamin Kaduk Reviewed-by: D Brashear (cherry picked from commit 69705782318226806b6a2b092ec4b63d594d184c) Change-Id: I44ea83c6b7da68fe0c5c8959d2c8eafb6e942d4b Reviewed-on: http://gerrit.openafs.org/11207 Tested-by: BuildBot Reviewed-by: Perry Ruiter Reviewed-by: Andrew Deason Reviewed-by: Benjamin Kaduk Reviewed-by: Stephan Wiesand --- src/afs/afs_call.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/afs/afs_call.c b/src/afs/afs_call.c index f15aa8076..196bc849c 100644 --- a/src/afs/afs_call.c +++ b/src/afs/afs_call.c @@ -1047,10 +1047,6 @@ afs_syscall_call(long parm, long parm2, long parm3, code = 0; AFS_COPYINSTR(AFSKPTR(parm2), tbuffer, AFS_SMALLOCSIZ, &bufferSize, code); - if (code) { - osi_FreeSmallSpace(tbuffer); - goto out; - } if (!code) { tbuffer[AFS_SMALLOCSIZ - 1] = '\0'; /* null-terminate the name */ /* We have the cache dir copied in. Call the cache init routine */ -- 2.39.5