Do initialise fp to NULL explicitly, the last comment wasn't enough.
authorJoerg Sonnenberger <joerg@dragonflybsd.org>
Sat, 5 Feb 2005 22:20:20 +0000 (22:20 +0000)
committerJoerg Sonnenberger <joerg@dragonflybsd.org>
Sat, 5 Feb 2005 22:20:20 +0000 (22:20 +0000)
The first conditional can fail and the variable points to some random
place.

sys/kern/uipc_syscalls.c

index 52e0906..b3fa2b4 100644 (file)
@@ -35,7 +35,7 @@
  *
  *     @(#)uipc_syscalls.c     8.4 (Berkeley) 2/21/94
  * $FreeBSD: src/sys/kern/uipc_syscalls.c,v 1.65.2.17 2003/04/04 17:11:16 tegge Exp $
- * $DragonFly: src/sys/kern/uipc_syscalls.c,v 1.47 2005/01/12 08:08:09 hsu Exp $
+ * $DragonFly: src/sys/kern/uipc_syscalls.c,v 1.48 2005/02/05 22:20:20 joerg Exp $
  */
 
 #include "opt_ktrace.h"
@@ -1432,7 +1432,7 @@ kern_sendfile(struct vnode *vp, int sfd, off_t offset, size_t nbytes,
        struct proc *p = td->td_proc;
        struct vm_object *obj;
        struct socket *so;
-       struct file *fp;
+       struct file *fp = NULL;
        struct mbuf *m;
        struct sf_buf *sf;
        struct sfbuf_mref *sfm;