mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	xdp: fix uninitialized 'err' variable
Smatch caught an uninitialized variable error which GCC seems
to miss.
Fixes: a25717d2b6 ("xdp: support simultaneous driver and hw XDP attachment")
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
			
			
This commit is contained in:
		
							parent
							
								
									e146471f58
								
							
						
					
					
						commit
						202aabe84a
					
				
					 1 changed files with 9 additions and 6 deletions
				
			
		| 
						 | 
					@ -1414,14 +1414,17 @@ static int rtnl_xdp_fill(struct sk_buff *skb, struct net_device *dev)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	prog_id = 0;
 | 
						prog_id = 0;
 | 
				
			||||||
	mode = XDP_ATTACHED_NONE;
 | 
						mode = XDP_ATTACHED_NONE;
 | 
				
			||||||
	if (rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_SKB,
 | 
						err = rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_SKB,
 | 
				
			||||||
				IFLA_XDP_SKB_PROG_ID, rtnl_xdp_prog_skb))
 | 
									  IFLA_XDP_SKB_PROG_ID, rtnl_xdp_prog_skb);
 | 
				
			||||||
 | 
						if (err)
 | 
				
			||||||
		goto err_cancel;
 | 
							goto err_cancel;
 | 
				
			||||||
	if (rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_DRV,
 | 
						err = rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_DRV,
 | 
				
			||||||
				IFLA_XDP_DRV_PROG_ID, rtnl_xdp_prog_drv))
 | 
									  IFLA_XDP_DRV_PROG_ID, rtnl_xdp_prog_drv);
 | 
				
			||||||
 | 
						if (err)
 | 
				
			||||||
		goto err_cancel;
 | 
							goto err_cancel;
 | 
				
			||||||
	if (rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_HW,
 | 
						err = rtnl_xdp_report_one(skb, dev, &prog_id, &mode, XDP_ATTACHED_HW,
 | 
				
			||||||
				IFLA_XDP_HW_PROG_ID, rtnl_xdp_prog_hw))
 | 
									  IFLA_XDP_HW_PROG_ID, rtnl_xdp_prog_hw);
 | 
				
			||||||
 | 
						if (err)
 | 
				
			||||||
		goto err_cancel;
 | 
							goto err_cancel;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	err = nla_put_u8(skb, IFLA_XDP_ATTACHED, mode);
 | 
						err = nla_put_u8(skb, IFLA_XDP_ATTACHED, mode);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue