]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
afs: Delete unneeded duplicate code
authorPerry Ruiter <pruiter@sinenomine.net>
Fri, 30 May 2014 02:24:24 +0000 (19:24 -0700)
committerStephan Wiesand <stephan.wiesand@desy.de>
Thu, 19 Jun 2014 12:25:25 +0000 (08:25 -0400)
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 <buildbot@rampaginggeek.com>
Reviewed-by: Chas Williams - CONTRACTOR <chas@cmf.nrl.navy.mil>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: D Brashear <shadow@your-file-system.com>
(cherry picked from commit 69705782318226806b6a2b092ec4b63d594d184c)

Change-Id: I44ea83c6b7da68fe0c5c8959d2c8eafb6e942d4b
Reviewed-on: http://gerrit.openafs.org/11207
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Perry Ruiter <pruiter@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/afs/afs_call.c

index f15aa8076bf53a313d07184986bf7f741e8ac2c3..196bc849ce1b17fd76479bd5af7a9b52d78a74f3 100644 (file)
@@ -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 */