diff options
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/bcast.c | 6 | ||||
| -rw-r--r-- | net/tipc/config.c | 32 | ||||
| -rw-r--r-- | net/tipc/dbg.c | 3 | ||||
| -rw-r--r-- | net/tipc/name_distr.c | 10 | ||||
| -rw-r--r-- | net/tipc/node.c | 12 | ||||
| -rw-r--r-- | net/tipc/subscr.c | 3 | 
6 files changed, 24 insertions, 42 deletions
| diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index 1bb75703f38..730c5c47ed8 100644 --- a/net/tipc/bcast.c +++ b/net/tipc/bcast.c @@ -774,8 +774,8 @@ int tipc_bclink_set_queue_limits(u32 limit)  int tipc_bclink_init(void)  { -	bcbearer = kmalloc(sizeof(*bcbearer), GFP_ATOMIC); -	bclink = kmalloc(sizeof(*bclink), GFP_ATOMIC); +	bcbearer = kzalloc(sizeof(*bcbearer), GFP_ATOMIC); +	bclink = kzalloc(sizeof(*bclink), GFP_ATOMIC);  	if (!bcbearer || !bclink) {   nomem:  	 	warn("Multicast link creation failed, no memory\n"); @@ -786,14 +786,12 @@ int tipc_bclink_init(void)  		return -ENOMEM;  	} -	memset(bcbearer, 0, sizeof(struct bcbearer));  	INIT_LIST_HEAD(&bcbearer->bearer.cong_links);  	bcbearer->bearer.media = &bcbearer->media;  	bcbearer->media.send_msg = tipc_bcbearer_send;  	sprintf(bcbearer->media.name, "tipc-multicast");  	bcl = &bclink->link; -	memset(bclink, 0, sizeof(struct bclink));  	INIT_LIST_HEAD(&bcl->waiting_ports);  	bcl->next_out_no = 1;  	spin_lock_init(&bclink->node.lock); diff --git a/net/tipc/config.c b/net/tipc/config.c index ed1351ed05e..458a2c46cef 100644 --- a/net/tipc/config.c +++ b/net/tipc/config.c @@ -107,7 +107,7 @@ int tipc_cfg_append_tlv(struct sk_buff *buf, int tlv_type,  struct sk_buff *tipc_cfg_reply_unsigned_type(u16 tlv_type, u32 value)  {  	struct sk_buff *buf; -	u32 value_net; +	__be32 value_net;  	buf = tipc_cfg_reply_alloc(TLV_SPACE(sizeof(value)));  	if (buf) { @@ -284,8 +284,7 @@ static struct sk_buff *cfg_set_own_addr(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_NET_ADDR))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	addr = *(u32 *)TLV_DATA(req_tlv_area); -	addr = ntohl(addr); +	addr = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (addr == tipc_own_addr)  		return tipc_cfg_reply_none();  	if (!tipc_addr_node_valid(addr)) @@ -319,8 +318,7 @@ static struct sk_buff *cfg_set_remote_mng(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	tipc_remote_management = (value != 0);  	return tipc_cfg_reply_none();  } @@ -332,8 +330,7 @@ static struct sk_buff *cfg_set_max_publications(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value != delimit(value, 1, 65535))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (max publications must be 1-65535)"); @@ -348,8 +345,7 @@ static struct sk_buff *cfg_set_max_subscriptions(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value != delimit(value, 1, 65535))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (max subscriptions must be 1-65535"); @@ -363,8 +359,7 @@ static struct sk_buff *cfg_set_max_ports(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value == tipc_max_ports)  		return tipc_cfg_reply_none();  	if (value != delimit(value, 127, 65535)) @@ -383,8 +378,7 @@ static struct sk_buff *cfg_set_max_zones(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value == tipc_max_zones)  		return tipc_cfg_reply_none();  	if (value != delimit(value, 1, 255)) @@ -403,8 +397,7 @@ static struct sk_buff *cfg_set_max_clusters(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value != delimit(value, 1, 1))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (max clusters fixed at 1)"); @@ -417,8 +410,7 @@ static struct sk_buff *cfg_set_max_nodes(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value == tipc_max_nodes)  		return tipc_cfg_reply_none();  	if (value != delimit(value, 8, 2047)) @@ -437,8 +429,7 @@ static struct sk_buff *cfg_set_max_slaves(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value != 0)  		return tipc_cfg_reply_error_string(TIPC_CFG_NOT_SUPPORTED  						   " (max secondary nodes fixed at 0)"); @@ -451,8 +442,7 @@ static struct sk_buff *cfg_set_netid(void)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value == tipc_net_id)  		return tipc_cfg_reply_none();  	if (value != delimit(value, 1, 9999)) diff --git a/net/tipc/dbg.c b/net/tipc/dbg.c index d8af4c28695..627f99b7afd 100644 --- a/net/tipc/dbg.c +++ b/net/tipc/dbg.c @@ -393,8 +393,7 @@ struct sk_buff *tipc_log_resize(const void *req_tlv_area, int req_tlv_space)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_UNSIGNED))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	value = *(u32 *)TLV_DATA(req_tlv_area); -	value = ntohl(value); +	value = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (value != delimit(value, 0, 32768))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (log size must be 0-32768)"); diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c index 03bd659c43c..7bf87cb26ef 100644 --- a/net/tipc/name_distr.c +++ b/net/tipc/name_distr.c @@ -66,11 +66,11 @@   */  struct distr_item { -	u32 type; -	u32 lower; -	u32 upper; -	u32 ref; -	u32 key; +	__be32 type; +	__be32 lower; +	__be32 upper; +	__be32 ref; +	__be32 key;  };  /** diff --git a/net/tipc/node.c b/net/tipc/node.c index 886bda5e88d..4111a31def7 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c @@ -60,7 +60,7 @@ struct node *tipc_node_create(u32 addr)  	struct node *n_ptr;          struct node **curr_node; -	n_ptr = kmalloc(sizeof(*n_ptr),GFP_ATOMIC); +	n_ptr = kzalloc(sizeof(*n_ptr),GFP_ATOMIC);  	if (!n_ptr) {  		warn("Node creation failed, no memory\n");  		return NULL; @@ -75,7 +75,6 @@ struct node *tipc_node_create(u32 addr)  		return NULL;  	} -	memset(n_ptr, 0, sizeof(*n_ptr));  	n_ptr->addr = addr;                  spin_lock_init(&n_ptr->lock);  	INIT_LIST_HEAD(&n_ptr->nsub); @@ -597,8 +596,7 @@ struct sk_buff *tipc_node_get_nodes(const void *req_tlv_area, int req_tlv_space)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_NET_ADDR))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	domain = *(u32 *)TLV_DATA(req_tlv_area); -	domain = ntohl(domain); +	domain = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (!tipc_addr_domain_valid(domain))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (network address)"); @@ -642,8 +640,7 @@ struct sk_buff *tipc_node_get_links(const void *req_tlv_area, int req_tlv_space)  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_NET_ADDR))  		return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR); -	domain = *(u32 *)TLV_DATA(req_tlv_area); -	domain = ntohl(domain); +	domain = ntohl(*(__be32 *)TLV_DATA(req_tlv_area));  	if (!tipc_addr_domain_valid(domain))  		return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE  						   " (network address)"); @@ -664,8 +661,7 @@ struct sk_buff *tipc_node_get_links(const void *req_tlv_area, int req_tlv_space)  	/* Add TLV for broadcast link */ -        link_info.dest = tipc_own_addr & 0xfffff00; -	link_info.dest = htonl(link_info.dest); +        link_info.dest = htonl(tipc_own_addr & 0xfffff00);          link_info.up = htonl(1);          sprintf(link_info.str, tipc_bclink_name);  	tipc_cfg_append_tlv(buf, TIPC_TLV_LINK_INFO, &link_info, sizeof(link_info)); diff --git a/net/tipc/subscr.c b/net/tipc/subscr.c index 7a918f12a5d..ddade7388aa 100644 --- a/net/tipc/subscr.c +++ b/net/tipc/subscr.c @@ -350,7 +350,7 @@ static void subscr_subscribe(struct tipc_subscr *s,  	/* Allocate subscription object */ -	sub = kmalloc(sizeof(*sub), GFP_ATOMIC); +	sub = kzalloc(sizeof(*sub), GFP_ATOMIC);  	if (!sub) {  		warn("Subscription rejected, no memory\n");  		subscr_terminate(subscriber); @@ -359,7 +359,6 @@ static void subscr_subscribe(struct tipc_subscr *s,  	/* Initialize subscription object */ -	memset(sub, 0, sizeof(*sub));  	sub->seq.type = htohl(s->seq.type, subscriber->swap);  	sub->seq.lower = htohl(s->seq.lower, subscriber->swap);  	sub->seq.upper = htohl(s->seq.upper, subscriber->swap); | 
