• Julia Lawall's avatar
    drivers/isdn: Drop unnecessary NULL test · 3cad3da3
    Julia Lawall authored
    The result of container_of should not be NULL.  In particular, in this case
    the argument to the enclosing function has passed though INIT_WORK, which
    dereferences it, implying that its container cannot be NULL.
    
    A simplified version of the semantic patch that makes this change is as
    follows:
    (http://www.emn.fr/x-info/coccinelle/)
    
    // <smpl>
    @@
    identifier fn,work,x,fld;
    type T;
    expression E1,E2;
    statement S;
    @@
    
    static fn(struct work_struct *work) {
      ... when != work = E1
      x = container_of(work,T,fld)
      ... when != x = E2
    - if (x == NULL) S
      ...
    }
    // </smpl>
    Signed-off-by: default avatarJulia Lawall <julia@diku.dk>
    Signed-off-by: default avatarKarsten Keil <keil@b1-systems.de>
    3cad3da3
icc.c 18 KB