* Use 'test.sh'.

* Use 'test_brkm' instead of 'tst_resm TBROK'.

* Use 'tst_tmpdir' and 'tst_rmdir'.

* Delete some useless variable value.

* Delete some unuseful comments.

* Some cleanup.

Signed-off-by: Zeng Linggang <zenglg...@cn.fujitsu.com>
---
 .../security/integrity/ima/tests/ima_setup.sh      | 85 +++-------------------
 1 file changed, 12 insertions(+), 73 deletions(-)

diff --git a/testcases/kernel/security/integrity/ima/tests/ima_setup.sh 
b/testcases/kernel/security/integrity/ima/tests/ima_setup.sh
index 4013e1f..0ff38d2 100755
--- a/testcases/kernel/security/integrity/ima/tests/ima_setup.sh
+++ b/testcases/kernel/security/integrity/ima/tests/ima_setup.sh
@@ -14,8 +14,8 @@
 ## for more details.                                                          
##
 ##                                                                            
##
 ## You should have received a copy of the GNU General Public License          
##
-## along with this program;  if not, write to the Free Software               
##
-## Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
USA    ##
+## along with this program;  if not, write to the Free Software Foundation,   
##
+## Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA           
##
 ##                                                                            
##
 
################################################################################
 #
@@ -24,13 +24,10 @@
 # Description:  setup/cleanup routines for the integrity tests.
 #
 # Author:       Mimi Zohar, zo...@ibm.vnet.ibm.com
-#
-# Return        - zero on success
-#               - non zero on failure. return value from commands ($RC)
 
################################################################################
+. test.sh
 mount_sysfs()
 {
-
        SYSFS=$(mount 2>/dev/null | awk '$5 == "sysfs" { print $3 }')
        if [ "x$SYSFS" = x ] ; then
 
@@ -38,22 +35,17 @@ mount_sysfs()
 
                test -d $SYSFS || mkdir -p $SYSFS 2>/dev/null
                if [ $? -ne 0 ] ; then
-                       tst_resm TBROK "Failed to mkdir $SYSFS"
-                       return 1
+                       tst_brkm TBROK "Failed to mkdir $SYSFS"
                fi
                if ! mount -t sysfs sysfs $SYSFS 2>/dev/null ; then
-                       tst_resm TBROK "Failed to mount $SYSFS"
-                       return 1
+                       tst_brkm TBROK "Failed to mount $SYSFS"
                fi
 
        fi
-
-       return 0
 }
 
 mount_securityfs()
 {
-
        SECURITYFS=$(mount 2>/dev/null | awk '$5 == "securityfs" { print $3 }')
        if [ "x$SECURITYFS" = x ] ; then
 
@@ -61,87 +53,34 @@ mount_securityfs()
 
                test -d $SECURITYFS || mkdir -p $SECURITYFS 2>/dev/null
                if [ $? -ne 0 ] ; then
-                       tst_resm TBROK "Failed to mkdir $SECURITYFS"
-                       return 1
+                       tst_brkm TBROK "Failed to mkdir $SECURITYFS"
                fi
                if ! mount -t securityfs securityfs $SECURITYFS 2>/dev/null ; 
then
-                       tst_resm TBROK "Failed to mount $SECURITYFS"
-                       return 1
+                       tst_brkm TBROK "Failed to mount $SECURITYFS"
                fi
 
        fi
-
-       return 0
-
 }
 
 setup()
 {
-       export TST_TOTAL=1
-       export TCID="setup"
-        export TST_COUNT=0
-
-       LTPBIN=
-       LTPIMA=
-
-       trap "cleanup" 0
-       if [ -z "$TMPDIR" ]; then
-               LTPTMP=/tmp
-       else
-               LTPTMP=${TMPDIR}
-       fi
-       if [ -z "$LTPBIN" ]; then
-               LTPBIN=../../../../../bin
-               PATH=$PATH:$LTPBIN
-       fi
-
        tst_require_root
 
-       # create the temporary directory used by this testcase
-       LTPIMA=$LTPTMP/ima
-       umask 077
-       mkdir $LTPIMA > /dev/null 2>&1 || RC=$?
-       if [ $RC -ne 0 ]; then
-               tst_resm TBROK "Unable to create temporary directory"
-               return $RC
-       fi
+       tst_tmpdir
 
-       # mount sysfs if it is not already mounted
-       mount_sysfs || RC=$?
-       if [ $RC -ne 0 ]; then
-               tst_resm TBROK "Cannot mount sysfs"
-               return $RC
-       fi
+       mount_sysfs
 
        # mount securityfs if it is not already mounted
-       mount_securityfs || RC=$?
-       if [ $RC -ne 0 ]; then
-               tst_resm TBROK "Cannot mount securityfs"
-               return $RC
-       fi
-
-       mount
+       mount_securityfs
 
        # IMA must be configured in the kernel
        IMA_DIR=$SECURITYFS/ima
        if [ ! -d "$IMA_DIR" ]; then
-               tst_resm TCONF "IMA not enabled in kernel"
-               RC=1
+               tst_brkm TCONF "IMA not enabled in kernel"
        fi
-       return $RC
 }
 
-# Function:     cleanup
-#
-# Description   - remove temporary files and directories.
-#
-# Return        - zero on success
-#               - non zero on failure. return value from commands ($RC)
 cleanup()
 {
-       tst_resm TINFO "CLEAN: removing $LTPIMA"
-       rm -rf "$LTPIMA" || RC=$?
-       return $RC
+       tst_rmdir
 }
-
-. cmdlib.sh
-- 
1.9.3




------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to