Inter-revision diff: patch 19

Comparing v33 (message) to v24 (message)

--- v33
+++ v24
@@ -8,7 +8,7 @@
 
 Reviewed-by: Kees Cook <keescook@chromium.org>
 Reviewed-by: John Johansen <john.johansen@canonical.com>
-Acked-by: Stephen Smalley <stephen.smalley.work@gmail.com>
+Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
 Acked-by: Paul Moore <paul@paul-moore.com>
 Signed-off-by: Casey Schaufler <casey@schaufler-ca.com>
 Cc: netdev@vger.kernel.org
@@ -69,7 +69,7 @@
  {
  	return -ENOSYS;
 diff --git a/net/ipv4/cipso_ipv4.c b/net/ipv4/cipso_ipv4.c
-index 62d5f99760aa..bb9c900da6b0 100644
+index 471d33a0d095..1ac343d02b58 100644
 --- a/net/ipv4/cipso_ipv4.c
 +++ b/net/ipv4/cipso_ipv4.c
 @@ -106,15 +106,17 @@ int cipso_v4_rbm_strictvalid = 1;
@@ -98,7 +98,7 @@
  
  /*
   * Helper Functions
-@@ -1460,7 +1462,11 @@ static int cipso_v4_gentag_loc(const struct cipso_v4_doi *doi_def,
+@@ -1469,7 +1471,11 @@ static int cipso_v4_gentag_loc(const struct cipso_v4_doi *doi_def,
  
  	buffer[0] = CIPSO_V4_TAG_LOCAL;
  	buffer[1] = CIPSO_V4_TAG_LOC_BLEN;
@@ -111,7 +111,7 @@
  
  	return CIPSO_V4_TAG_LOC_BLEN;
  }
-@@ -1480,7 +1486,7 @@ static int cipso_v4_parsetag_loc(const struct cipso_v4_doi *doi_def,
+@@ -1489,7 +1495,7 @@ static int cipso_v4_parsetag_loc(const struct cipso_v4_doi *doi_def,
  				 const unsigned char *tag,
  				 struct netlbl_lsm_secattr *secattr)
  {
@@ -121,7 +121,7 @@
  
  	return 0;
 diff --git a/net/netlabel/netlabel_kapi.c b/net/netlabel/netlabel_kapi.c
-index beb0e573266d..158bab993e32 100644
+index 5e1239cef000..bbfaff539416 100644
 --- a/net/netlabel/netlabel_kapi.c
 +++ b/net/netlabel/netlabel_kapi.c
 @@ -196,7 +196,7 @@ int netlbl_cfg_unlbl_map_add(const char *domain,
@@ -152,7 +152,7 @@
  
  /**
 diff --git a/net/netlabel/netlabel_unlabeled.c b/net/netlabel/netlabel_unlabeled.c
-index 46706889a6f7..3aab71ba3841 100644
+index aa53a94115f4..3befe0738d31 100644
 --- a/net/netlabel/netlabel_unlabeled.c
 +++ b/net/netlabel/netlabel_unlabeled.c
 @@ -66,7 +66,7 @@ struct netlbl_unlhsh_tbl {
@@ -283,10 +283,10 @@
  
  	spin_lock(&netlbl_unlhsh_lock);
  	list_entry = netlbl_af4list_remove(addr->s_addr, mask->s_addr,
-@@ -496,13 +490,8 @@ static int netlbl_unlhsh_remove_addr4(struct net *net,
- 					  (dev != NULL ? dev->name : NULL),
+@@ -497,13 +491,8 @@ static int netlbl_unlhsh_remove_addr4(struct net *net,
  					  addr->s_addr, mask->s_addr);
- 		dev_put(dev);
+ 		if (dev != NULL)
+ 			dev_put(dev);
 -		/* lsmblob_init() puts entry->secid into all of the secids
 -		 * in blob. security_secid_to_secctx() will know which
 -		 * security module to use to create the secctx.  */
@@ -298,7 +298,7 @@
  			audit_log_format(audit_buf, " sec_obj=%s",
  					 context.context);
  			security_release_secctx(&context);
-@@ -543,7 +532,6 @@ static int netlbl_unlhsh_remove_addr6(struct net *net,
+@@ -544,7 +533,6 @@ static int netlbl_unlhsh_remove_addr6(struct net *net,
  	struct audit_buffer *audit_buf;
  	struct net_device *dev;
  	struct lsmcontext context;
@@ -306,10 +306,10 @@
  
  	spin_lock(&netlbl_unlhsh_lock);
  	list_entry = netlbl_af6list_remove(addr, mask, &iface->addr6_list);
-@@ -561,13 +549,8 @@ static int netlbl_unlhsh_remove_addr6(struct net *net,
- 					  (dev != NULL ? dev->name : NULL),
+@@ -563,13 +551,8 @@ static int netlbl_unlhsh_remove_addr6(struct net *net,
  					  addr, mask);
- 		dev_put(dev);
+ 		if (dev != NULL)
+ 			dev_put(dev);
 -		/* lsmblob_init() puts entry->secid into all of the secids
 -		 * in blob. security_secid_to_secctx() will know which
 -		 * security module to use to create the secctx.  */
@@ -321,7 +321,7 @@
  			audit_log_format(audit_buf, " sec_obj=%s",
  					 context.context);
  			security_release_secctx(&context);
-@@ -921,14 +904,8 @@ static int netlbl_unlabel_staticadd(struct sk_buff *skb,
+@@ -923,14 +906,8 @@ static int netlbl_unlabel_staticadd(struct sk_buff *skb,
  	if (ret_val != 0)
  		return ret_val;
  
@@ -338,7 +338,7 @@
  }
  
  /**
-@@ -975,11 +952,8 @@ static int netlbl_unlabel_staticadddef(struct sk_buff *skb,
+@@ -977,11 +954,8 @@ static int netlbl_unlabel_staticadddef(struct sk_buff *skb,
  	if (ret_val != 0)
  		return ret_val;
  
@@ -352,7 +352,7 @@
  }
  
  /**
-@@ -1091,8 +1065,7 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
+@@ -1093,8 +1067,7 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
  	struct net_device *dev;
  	struct lsmcontext context;
  	void *data;
@@ -362,7 +362,7 @@
  
  	data = genlmsg_put(cb_arg->skb, NETLINK_CB(cb_arg->nl_cb->skb).portid,
  			   cb_arg->seq, &netlbl_unlabel_gnl_family,
-@@ -1130,7 +1103,7 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
+@@ -1132,7 +1105,7 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
  		if (ret_val != 0)
  			goto list_cb_failure;
  
@@ -371,7 +371,7 @@
  	} else {
  		ret_val = nla_put_in6_addr(cb_arg->skb,
  					   NLBL_UNLABEL_A_IPV6ADDR,
-@@ -1144,14 +1117,10 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
+@@ -1146,14 +1119,10 @@ static int netlbl_unlabel_staticlist_gen(u32 cmd,
  		if (ret_val != 0)
  			goto list_cb_failure;
  
@@ -388,7 +388,7 @@
  	if (ret_val != 0)
  		goto list_cb_failure;
  	ret_val = nla_put(cb_arg->skb,
-@@ -1510,7 +1479,7 @@ int netlbl_unlabel_getattr(const struct sk_buff *skb,
+@@ -1512,7 +1481,7 @@ int netlbl_unlabel_getattr(const struct sk_buff *skb,
  					      &iface->addr4_list);
  		if (addr4 == NULL)
  			goto unlabel_getattr_nolabel;
@@ -397,7 +397,7 @@
  		break;
  	}
  #if IS_ENABLED(CONFIG_IPV6)
-@@ -1523,7 +1492,7 @@ int netlbl_unlabel_getattr(const struct sk_buff *skb,
+@@ -1525,7 +1494,7 @@ int netlbl_unlabel_getattr(const struct sk_buff *skb,
  					      &iface->addr6_list);
  		if (addr6 == NULL)
  			goto unlabel_getattr_nolabel;
@@ -420,12 +420,12 @@
  int netlbl_unlhsh_remove(struct net *net,
  			 const char *dev_name,
 diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
-index 3469ffe195e6..7b9cb4d263c0 100644
+index 7bc9a043a30a..60e35d31cc4c 100644
 --- a/security/selinux/hooks.c
 +++ b/security/selinux/hooks.c
-@@ -7020,7 +7020,7 @@ static int selinux_uring_sqpoll(void)
+@@ -6952,7 +6952,7 @@ static int selinux_perf_event_write(struct perf_event *event)
  }
- #endif /* CONFIG_IO_URING */
+ #endif
  
 -static struct lsm_id selinux_lsmid __lsm_ro_after_init = {
 +struct lsm_id selinux_lsmid __lsm_ro_after_init = {
@@ -433,7 +433,7 @@
  	.slot = LSMBLOB_NEEDED
  };
 diff --git a/security/selinux/include/security.h b/security/selinux/include/security.h
-index ac0ece01305a..9f856f2cd277 100644
+index 3cc8bab31ea8..6a40b47307ca 100644
 --- a/security/selinux/include/security.h
 +++ b/security/selinux/include/security.h
 @@ -73,6 +73,7 @@
@@ -445,10 +445,10 @@
  /*
   * type_datum properties
 diff --git a/security/selinux/netlabel.c b/security/selinux/netlabel.c
-index 800ab4b4239e..0b8f99703462 100644
+index 6a94b31b5472..d8d7603ab14e 100644
 --- a/security/selinux/netlabel.c
 +++ b/security/selinux/netlabel.c
-@@ -109,7 +109,7 @@ static struct netlbl_lsm_secattr *selinux_netlbl_sock_getattr(
+@@ -108,7 +108,7 @@ static struct netlbl_lsm_secattr *selinux_netlbl_sock_getattr(
  		return NULL;
  
  	if ((secattr->flags & NETLBL_SECATTR_SECID) &&
@@ -458,10 +458,10 @@
  
  	return NULL;
 diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c
-index 8e92af7dd284..23a45c9dcf04 100644
+index 597b79703584..a18339bc883a 100644
 --- a/security/selinux/ss/services.c
 +++ b/security/selinux/ss/services.c
-@@ -3899,7 +3899,7 @@ int security_netlbl_secattr_to_sid(struct selinux_state *state,
+@@ -3787,7 +3787,7 @@ int security_netlbl_secattr_to_sid(struct selinux_state *state,
  	if (secattr->flags & NETLBL_SECATTR_CACHE)
  		*sid = *(u32 *)secattr->cache->data;
  	else if (secattr->flags & NETLBL_SECATTR_SECID)
@@ -470,7 +470,7 @@
  	else if (secattr->flags & NETLBL_SECATTR_MLS_LVL) {
  		rc = -EIDRM;
  		ctx = sidtab_search(sidtab, SECINITSID_NETMSG);
-@@ -3977,7 +3977,7 @@ int security_netlbl_sid_to_secattr(struct selinux_state *state,
+@@ -3863,7 +3863,7 @@ int security_netlbl_sid_to_secattr(struct selinux_state *state,
  	if (secattr->domain == NULL)
  		goto out;
  
@@ -480,22 +480,22 @@
  	mls_export_netlbl_lvl(policydb, ctx, secattr);
  	rc = mls_export_netlbl_cat(policydb, ctx, secattr);
 diff --git a/security/smack/smack.h b/security/smack/smack.h
-index ef9d0b7b1954..ac79313ea95d 100644
+index 0f8d0feb89a4..b06fc332a1f9 100644
 --- a/security/smack/smack.h
 +++ b/security/smack/smack.h
 @@ -303,6 +303,7 @@ int smack_populate_secattr(struct smack_known *skp);
   * Shared data.
   */
- extern int smack_enabled __initdata;
+ extern int smack_enabled;
 +extern struct lsm_id smack_lsmid;
  extern int smack_cipso_direct;
  extern int smack_cipso_mapped;
  extern struct smack_known *smack_net_ambient;
 diff --git a/security/smack/smack_access.c b/security/smack/smack_access.c
-index d2186e2757be..c6dcafe18912 100644
+index 7eabb448acab..fccd5da3014e 100644
 --- a/security/smack/smack_access.c
 +++ b/security/smack/smack_access.c
-@@ -524,7 +524,7 @@ int smack_populate_secattr(struct smack_known *skp)
+@@ -522,7 +522,7 @@ int smack_populate_secattr(struct smack_known *skp)
  {
  	int slen;
  
@@ -505,10 +505,10 @@
  	skp->smk_netlabel.cache = netlbl_secattr_cache_alloc(GFP_ATOMIC);
  	if (skp->smk_netlabel.cache != NULL) {
 diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
-index 3c1cf65cac87..46d81f638a2b 100644
+index ef04029dcdf1..03a1c40174d7 100644
 --- a/security/smack/smack_lsm.c
 +++ b/security/smack/smack_lsm.c
-@@ -3728,11 +3728,12 @@ static struct smack_known *smack_from_secattr(struct netlbl_lsm_secattr *sap,
+@@ -3721,11 +3721,12 @@ static struct smack_known *smack_from_secattr(struct netlbl_lsm_secattr *sap,
  	if ((sap->flags & NETLBL_SECATTR_CACHE) != 0)
  		return (struct smack_known *)sap->cache->data;
  
@@ -525,8 +525,8 @@
  
  	if ((sap->flags & NETLBL_SECATTR_MLS_LVL) != 0) {
  		/*
-@@ -4751,7 +4752,7 @@ struct lsm_blob_sizes smack_blob_sizes __lsm_ro_after_init = {
- 	.lbs_superblock = sizeof(struct superblock_smack),
+@@ -4699,7 +4700,7 @@ struct lsm_blob_sizes smack_blob_sizes __lsm_ro_after_init = {
+ 	.lbs_sock = sizeof(struct socket_smack),
  };
  
 -static struct lsm_id smack_lsmid __lsm_ro_after_init = {
@@ -535,10 +535,10 @@
  	.slot = LSMBLOB_NEEDED
  };
 diff --git a/security/smack/smackfs.c b/security/smack/smackfs.c
-index 658eab05599e..13c2fa728054 100644
+index 5d44b7d258ef..ad946ccf5023 100644
 --- a/security/smack/smackfs.c
 +++ b/security/smack/smackfs.c
-@@ -1143,6 +1143,7 @@ static void smk_net4addr_insert(struct smk_net4addr *new)
+@@ -1140,6 +1140,7 @@ static void smk_net4addr_insert(struct smk_net4addr *new)
  static ssize_t smk_write_net4addr(struct file *file, const char __user *buf,
  				size_t count, loff_t *ppos)
  {
@@ -546,7 +546,7 @@
  	struct smk_net4addr *snp;
  	struct sockaddr_in newname;
  	char *smack;
-@@ -1274,10 +1275,13 @@ static ssize_t smk_write_net4addr(struct file *file, const char __user *buf,
+@@ -1271,10 +1272,13 @@ static ssize_t smk_write_net4addr(struct file *file, const char __user *buf,
  	 * this host so that incoming packets get labeled.
  	 * but only if we didn't get the special CIPSO option
  	 */
@@ -564,5 +564,5 @@
  	if (rc == 0)
  		rc = count;
 -- 
-2.31.1
+2.25.4
 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help