This is just a guess. include/net/pkt_act.h: In function `tcf_hash_init': include/net/pkt_act.h:282: warning: control reaches end of non-void function I've never before seen so many coding style dorkinesses in such a small function, btw. Was this designed as a "how not to"? Signed-off-by: Andrew Morton --- 25-akpm/include/net/pkt_act.h | 1 + 1 files changed, 1 insertion(+) diff -puN include/net/pkt_act.h~pkt_act-fix include/net/pkt_act.h --- 25/include/net/pkt_act.h~pkt_act-fix 2004-09-02 23:57:37.662596864 -0700 +++ 25-akpm/include/net/pkt_act.h 2004-09-02 23:57:37.665596408 -0700 @@ -279,6 +279,7 @@ tcf_hash_init(struct tc_st *parm, struct if (NULL == p) { return tcf_hash_create(parm, est, a, size, ovr, bind); } + return NULL; } #endif _