On Wed, 29 Jul 2015, Dmitry Torokhov wrote: > > Another and less intrusive fix would be: > > > > char *name = dev->battery->desc->name; > > struct power_supply_desc *psy_desc = dev->battery->desc; > > power_supply_unregister(dev->battery); > > kfree(name); > > kfree(psy_desc); > > I would much rather prefer this to the other version as it does not > leave memory hanging around, potentially indefinitely, but ultimately it > is up to Jiri.
I must have been in some broken state of mind when applying the original one, thanks a lot for catching my brainfart, Dmitry! Kryzstof, could you please send me properly formatted patch with the above, on top of your previous fix? Thanks. -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
