RepoMirrors/selinux
1
0
Fork 0
mirror of https://github.com/SELinuxProject/selinux synced 2025-03-31 07:56:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

seobject: use sorted() instead of .sort()

Browse Source 
2to3 wanted to wrap all the .keys() calls in list() but doing sorted()
directly is better.

Signed-off-by: Jason Zaman <jason@perfinion.com>
Signed-off-by: James Carter <jwcart2@tycho.nsa.gov>
This commit is contained in:
Jason Zaman 2016-08-05 02:33:55 +08:00 committed by James Carter
parent b51c39499b
commit 35d44f17af
1 changed files with 22 additions and 46 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

68
policycoreutils/semanage/seobject.py
View File

@ -716,21 +716,17 @@ class loginRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
l.append("-a -s %s -r '%s' %s" % (ddict[k][0], ddict[k][1], k))
return l
def list(self, heading=1, locallist=0):
ddict = self.get_all(locallist)
ldict = self.get_all_logins()
lkeys = ldict.keys()
keys = ddict.keys()
lkeys = sorted(ldict.keys())
keys = sorted(ddict.keys())
if len(keys) == 0 and len(lkeys) == 0:
return
keys.sort()
lkeys.sort()
if is_mls_enabled == 1:
if heading:
@ -988,18 +984,15 @@ class seluserRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
l.append("-a -L %s -r %s -R '%s' %s" % (ddict[k][1], ddict[k][2], ddict[k][3], k))
return l
def list(self, heading=1, locallist=0):
ddict = self.get_all(locallist)
keys = ddict.keys()
if len(keys) == 0:
if len(ddict) == 0:
return
keys.sort()
keys = sorted(ddict.keys())
if is_mls_enabled == 1:
if heading:
@ -1251,9 +1244,7 @@ class portRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
if k[0] == k[1]:
l.append("-a -t %s -p %s %s" % (ddict[k][0], k[2], k[0]))
else:
@ -1262,10 +1253,9 @@ class portRecords(semanageRecords):
def list(self, heading=1, locallist=0):
ddict = self.get_all_by_type(locallist)
keys = ddict.keys()
if len(keys) == 0:
if len(ddict) == 0:
return
keys.sort()
keys = sorted(ddict.keys())
if heading:
print("%-30s %-8s %s\n" % (_("SELinux Port Type"), _("Proto"), _("Port Number")))
@ -1488,18 +1478,15 @@ class nodeRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
l.append("-a -M %s -p %s -t %s %s" % (k[1], k[2], ddict[k][2], k[0]))
return l
def list(self, heading=1, locallist=0):
ddict = self.get_all(locallist)
keys = ddict.keys()
if len(keys) == 0:
if len(ddict) == 0:
return
keys.sort()
keys = sorted(ddict.keys())
if heading:
print("%-18s %-18s %-5s %-5s\n" % ("IP Address", "Netmask", "Protocol", "Context"))
@ -1679,18 +1666,15 @@ class interfaceRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
l.append("-a -t %s %s" % (ddict[k][2], k))
return l
def list(self, heading=1, locallist=0):
ddict = self.get_all(locallist)
keys = ddict.keys()
if len(keys) == 0:
if len(ddict) == 0:
return
keys.sort()
keys = sorted(ddict.keys())
if heading:
print("%-30s %s\n" % (_("SELinux Interface"), _("Context")))
@ -2031,9 +2015,7 @@ class fcontextRecords(semanageRecords):
def customized(self):
l = []
fcon_dict = self.get_all(True)
keys = fcon_dict.keys()
keys.sort()
for k in keys:
for k in sorted(fcon_dict.keys()):
if fcon_dict[k]:
l.append("-a -f %s -t %s '%s'" % (file_type_str_to_option[k[1]], fcon_dict[k][2], k[0]))
@ -2044,12 +2026,10 @@ class fcontextRecords(semanageRecords):
def list(self, heading=1, locallist=0):
fcon_dict = self.get_all(locallist)
keys = fcon_dict.keys()
if len(keys) != 0:
keys.sort()
if len(fcon_dict) != 0:
if heading:
print("%-50s %-18s %s\n" % (_("SELinux fcontext"), _("type"), _("Context")))
for k in keys:
for k in sorted(fcon_dict.keys()):
if fcon_dict[k]:
if is_mls_enabled:
print("%-50s %-18s %s:%s:%s:%s " % (k[0], k[1], fcon_dict[k][0], fcon_dict[k][1], fcon_dict[k][2], translate(fcon_dict[k][3], False)))
@ -2223,9 +2203,7 @@ class booleanRecords(semanageRecords):
def customized(self):
l = []
ddict = self.get_all(True)
keys = ddict.keys()
keys.sort()
for k in keys:
for k in sorted(ddict.keys()):
if ddict[k]:
l.append("-m -%s %s" % (ddict[k][2], k))
return l
@ -2234,18 +2212,16 @@ class booleanRecords(semanageRecords):
on_off = (_("off"), _("on"))
if use_file:
ddict = self.get_all(locallist)
keys = ddict.keys()
for k in keys:
for k in sorted(ddict.keys()):
if ddict[k]:
print("%s=%s" % (k, ddict[k][2]))
return
ddict = self.get_all(locallist)
keys = ddict.keys()
if len(keys) == 0:
if len(ddict) == 0:
return
if heading:
print("%-30s %s %s %s\n" % (_("SELinux boolean"), _("State"), _("Default"), _("Description")))
for k in keys:
for k in sorted(ddict.keys()):
if ddict[k]:
print("%-30s (%-5s,%5s) %s" % (k, on_off[selinux.security_get_boolean_active(k)], on_off[ddict[k][2]], self.get_desc(k)))