Search String: Display: Description: Sort:

Results:

References: [ +subject:/^(?:^\s*(re|sv|fwd|fw)[\[\]\d]*[:>-]+\s*)*\[PATCH\s+1\/2\]\s+fs\/xfs\:\s+Drop\s+unnecessary\s+NULL\s+test\s*$/: 3 ]

Total 3 documents matching your query.

1. [PATCH 1/2] fs/xfs: Drop unnecessary NULL test (score: 1)
Author: Julia Lawall <julia@xxxxxxx>
Date: Mon, 13 Jul 2009 12:10:05 +0200 (CEST)
The result of container_of should not be NULL. In particular, in this case the argument to the enclosing function has passed though INIT_DELAYED_WORK, which dereferences it, implying that its contain
/archives/xfs/2009-07/msg00112.html (7,303 bytes)

2. Re: [PATCH 1/2] fs/xfs: Drop unnecessary NULL test (score: 1)
Author: Olaf Weber <olaf@xxxxxxx>
Date: Tue, 14 Jul 2009 12:34:43 +0200
Given the defn of container_of() it seems clear that neither its input pointer nor its result should ever be NULL. In this particular case, there is a check for NULL and early exit in xfs_mru_cache_c
/archives/xfs/2009-07/msg00124.html (9,217 bytes)

3. Re: [PATCH 1/2] fs/xfs: Drop unnecessary NULL test (score: 1)
Author: Julia Lawall <julia@xxxxxxx>
Date: Tue, 14 Jul 2009 14:15:42 +0200 (CEST)
container_of just does pointer arithmetic. The result of that arithmetic can be NULL, ie if the argument was obtained from an expression of the form &x->f where x was NULL. But in principle, values s
/archives/xfs/2009-07/msg00125.html (8,391 bytes)


This search system is powered by Namazu