tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git 
master
head:   76cf65d1377f733af1e2a55233e3353ffa577f54
commit: f392a1846489720fc2e063d1210633b6cf4ec5a4 [2112/3285] net: phylink: 
provide phylink_validate_mask_caps() helper
config: microblaze-randconfig-m031-20221023
compiler: microblaze-linux-gcc (GCC) 12.1.0

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <[email protected]>
| Reported-by: Dan Carpenter <[email protected]>

New smatch warnings:
drivers/net/phy/phylink.c:588 phylink_validate_mask_caps() warn: variable 
dereferenced before check 'state' (see line 583)

Old smatch warnings:
drivers/net/phy/phylink.c:1104 phylink_change_inband_advert() error: we 
previously assumed 'pl->pcs' could be null (see line 1087)

vim +/state +588 drivers/net/phy/phylink.c

f392a184648972 Russell King (Oracle  2022-10-17  574) void 
phylink_validate_mask_caps(unsigned long *supported,
f392a184648972 Russell King (Oracle  2022-10-17  575)                           
struct phylink_link_state *state,
f392a184648972 Russell King (Oracle  2022-10-17  576)                           
unsigned long mac_capabilities)
34ae2c09d46a2d Russell King (Oracle  2021-11-15  577) {
34ae2c09d46a2d Russell King (Oracle  2021-11-15  578)   
__ETHTOOL_DECLARE_LINK_MODE_MASK(mask) = { 0, };
3e6eab8f3ef93c Sean Anderson         2022-09-20  579    unsigned long caps;
34ae2c09d46a2d Russell King (Oracle  2021-11-15  580) 
34ae2c09d46a2d Russell King (Oracle  2021-11-15  581)   
phylink_set_port_modes(mask);
34ae2c09d46a2d Russell King (Oracle  2021-11-15  582)   phylink_set(mask, 
Autoneg);
f392a184648972 Russell King (Oracle  2022-10-17 @583)   caps = 
phylink_get_capabilities(state->interface, mac_capabilities,
                                                                                
        ^^^^^^^^^^^^^^^^
b7e9294885b610 Sean Anderson         2022-09-20  584                            
        state->rate_matching);
                                                                                
        ^^^^^^^^^^^^^^^^^^^^
Unchecked dereferences

3e6eab8f3ef93c Sean Anderson         2022-09-20  585    
phylink_caps_to_linkmodes(mask, caps);
34ae2c09d46a2d Russell King (Oracle  2021-11-15  586) 
34ae2c09d46a2d Russell King (Oracle  2021-11-15  587)   linkmode_and(supported, 
supported, mask);
f392a184648972 Russell King (Oracle  2022-10-17 @588)   if (state)
                                                            ^^^^^
Check for NULL

34ae2c09d46a2d Russell King (Oracle  2021-11-15  589)           
linkmode_and(state->advertising, state->advertising, mask);
34ae2c09d46a2d Russell King (Oracle  2021-11-15  590) }

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to