lkml.org 
[lkml]   [2018]   [Dec]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 1/2] of: of_node_get()/of_node_put() nodes held in phandle cache
    On Fri, Dec 14, 2018 at 12:43 AM <frowand.list@gmail.com> wrote:
    >
    > From: Frank Rowand <frank.rowand@sony.com>
    >
    > The phandle cache contains struct device_node pointers. The refcount
    > of the pointers was not incremented while in the cache, allowing use
    > after free error after kfree() of the node. Add the proper increment
    > and decrement of the use count.

    Since we pre-populate the cache at boot, all the nodes will have a ref
    count and will never be freed unless we happen to repopulate the whole
    cache. That doesn't seem ideal. The node pointer is not "in use" just
    because it is in the cache.

    Rob

    \
     
     \ /
      Last update: 2018-12-14 18:16    [W:5.368 / U:0.148 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site