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 <akpm@osdl.org>
---

 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
_