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