Re: [linux-next:master 7918/13392] drivers/soc/microchip/mpfs-sys-controller.c:73:1-6: WARNING: invalid free of devm_ allocated data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Conor, Arnd,

On 17/03/2022 at 22:17, conor dooley wrote:
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   97add482e9033a9386baa8824fbea34d0cbfe663
commit: d0054a470c33902f5ae88835ed8a8ecc3cf8faa4 [7918/13392] soc: add microchip polarfire soc system controller
config: ia64-randconfig-c004-20220317 (https://download.01.org/0day-ci/archive/20220318/202203180259.lgIylRZV-lkp@xxxxxxxxx/config)
compiler: ia64-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>


cocci warnings: (new ones prefixed by >>)
drivers/soc/microchip/mpfs-sys-controller.c:73:1-6: WARNING: invalid free of devm_ allocated data

Silly mistake, should've changed this when the alloc was converted to
devm_.

@Nicolas, rest of this week is public holiday, but I'll try to
send a fix before the weekend. Should it go to you or to Arnd?

I cannot queue in my branches because otherwise it will conflict with the patch already taken by Arnd as an individual patch [1]:
"soc: microchip: make mpfs_sys_controller_put static"

What I propose is that we deal with this during the stabilization phase as a "fix" after 5.18-rc1 is released (in 2 weeks).

Is there a drawback to this approach in the sense that it can trigger other test robots?

This way, we won't have to disturb Arnd as I believe that his branches are all ready to be sent to Linus after this week-end. Additional benefit, you don't have to change your plans and enjoy St Patrick week-end ;-).

Best regards,
  Nicolas

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/commit/?h=for-next&id=17588e867e694e8a03234628145f7c4d738288e2


vim +73 drivers/soc/microchip/mpfs-sys-controller.c

     66
     67        static void mpfs_sys_controller_delete(struct kref *kref)
     68        {
     69                struct mpfs_sys_controller *sys_controller = container_of(kref, struct mpfs_sys_controller,
     70                                                       consumers);
     71
     72                mbox_free_channel(sys_controller->chan);
   > 73                kfree(sys_controller);
     74        }
     75

---
0-DAY CI Kernel Test Service
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx


--
Nicolas Ferre




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux