forked from mirrors/linux
		
	net: add more debug info in skb_checksum_help()
This is a followup of previous patch. Dumping the stack trace is a good start, but printing basic skb information is probably better. Signed-off-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									d7ea0d9df2
								
							
						
					
					
						commit
						eeee4b77dc
					
				
					 1 changed files with 6 additions and 4 deletions
				
			
		|  | @ -3266,15 +3266,17 @@ int skb_checksum_help(struct sk_buff *skb) | |||
| 
 | ||||
| 	offset = skb_checksum_start_offset(skb); | ||||
| 	ret = -EINVAL; | ||||
| 	if (WARN_ON_ONCE(offset >= skb_headlen(skb))) | ||||
| 	if (WARN_ON_ONCE(offset >= skb_headlen(skb))) { | ||||
| 		DO_ONCE_LITE(skb_dump, KERN_ERR, skb, false); | ||||
| 		goto out; | ||||
| 
 | ||||
| 	} | ||||
| 	csum = skb_checksum(skb, offset, skb->len - offset, 0); | ||||
| 
 | ||||
| 	offset += skb->csum_offset; | ||||
| 	if (WARN_ON_ONCE(offset + sizeof(__sum16) > skb_headlen(skb))) | ||||
| 	if (WARN_ON_ONCE(offset + sizeof(__sum16) > skb_headlen(skb))) { | ||||
| 		DO_ONCE_LITE(skb_dump, KERN_ERR, skb, false); | ||||
| 		goto out; | ||||
| 
 | ||||
| 	} | ||||
| 	ret = skb_ensure_writable(skb, offset + sizeof(__sum16)); | ||||
| 	if (ret) | ||||
| 		goto out; | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Eric Dumazet
						Eric Dumazet