2015-08-01 21:11 GMT+09:00 Jiri Kosina <jkosina@xxxxxxxx>: > 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? Of course, I'll send next version. Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html