diff mbox series

[COMMITTED] Remove redundant mention of SXID_ERASE

Message ID 1513279186-27463-1-git-send-email-siddhesh@sourceware.org
State New
Headers show
Series [COMMITTED] Remove redundant mention of SXID_ERASE | expand

Commit Message

Siddhesh Poyarekar Dec. 14, 2017, 7:19 p.m. UTC
SXID_ERASE is implicit for all environment variables.  Avoid
mentioning it in the tunables list; that way only the ones with
SXID_IGNORE remain prominent and mentioned.  TODO: we need to audit
each of those cases and drop them to SXID_ERASE wherever possible.

No change in generated code on x86_64, so committed as obvious.

Siddhesh
---
 ChangeLog            |  4 ++++
 elf/dl-tunables.list | 11 +----------
 2 files changed, 5 insertions(+), 10 deletions(-)
diff mbox series

Patch

diff --git a/ChangeLog b/ChangeLog
index 018edea..3ae1fd2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@ 
+2017-12-14  Siddhesh Poyarekar  <siddhesh@sourceware.org>
+
+	* elf/dl-tunables.list: Remove redundant SXID_ERASE.
+
 2017-12-14  Florian Weimer  <fweimer@redhat.com>
 
 	[BZ #22607]
diff --git a/elf/dl-tunables.list b/elf/dl-tunables.list
index ec0fe20..cf1cf4c 100644
--- a/elf/dl-tunables.list
+++ b/elf/dl-tunables.list
@@ -78,15 +78,12 @@  glibc {
     }
     tcache_max {
       type: SIZE_T
-      security_level: SXID_ERASE
     }
     tcache_count {
       type: SIZE_T
-      security_level: SXID_ERASE
     }
     tcache_unsorted_limit {
       type: SIZE_T
-      security_level: SXID_ERASE
     }
   }
   tune {
@@ -94,7 +91,7 @@  glibc {
       type: UINT_64
       env_alias: LD_HWCAP_MASK
       default: HWCAP_IMPORTANT
-      }
+    }
   }
 
   elision {
@@ -102,32 +99,26 @@  glibc {
       type: INT_32
       minval: 0
       maxval: 1
-      security_level: SXID_ERASE
     }
     skip_lock_busy {
       type: INT_32
       default: 3
-      security_level: SXID_ERASE
     }
     skip_lock_internal_abort {
       type: INT_32
       default: 3
-      security_level: SXID_ERASE
     }
     skip_lock_after_retries {
       type: INT_32
       default: 3
-      security_level: SXID_ERASE
     }
     tries {
       type: INT_32
       default: 3
-      security_level: SXID_ERASE
     }
     skip_trylock_internal_abort {
       type: INT_32
       default: 3
-      security_level: SXID_ERASE
     }
   }
 }