[src/trunk]: src/sys/dev/ieee1394 May be easy to understand 'pay_len > 0' mor...

[kiyohara <kiyohara%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/f089f59837ee
branches:  trunk
changeset: 754848:f089f59837ee
user:      kiyohara <kiyohara%NetBSD.org@localhost>
date:      Fri May 14 12:25:19 2010 +0000

description:
May be easy to understand 'pay_len > 0' more than 'paylaod != NULL'.

diffstat:

 sys/dev/ieee1394/firewire.c |  8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diffs (30 lines):

diff -r d5f2ca0db108 -r f089f59837ee sys/dev/ieee1394/firewire.c
--- a/sys/dev/ieee1394/firewire.c       Fri May 14 12:10:07 2010 +0000
+++ b/sys/dev/ieee1394/firewire.c       Fri May 14 12:25:19 2010 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: firewire.c,v 1.31 2010/05/10 12:17:32 kiyohara Exp $   */
+/*     $NetBSD: firewire.c,v 1.32 2010/05/14 12:25:19 kiyohara Exp $   */
 /*-
  * Copyright (c) 2003 Hidetoshi Shimokawa
  * Copyright (c) 1998-2002 Katsushi Kobayashi and Hidetoshi Shimokawa
@@ -37,7 +37,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: firewire.c,v 1.31 2010/05/10 12:17:32 kiyohara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: firewire.c,v 1.32 2010/05/14 12:25:19 kiyohara Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -920,9 +920,9 @@
                return;
        }
        fw_xfer_unload(xfer);
-       if (xfer->send.payload != NULL)
+       if (xfer->send.pay_len > 0)
                kmem_free(xfer->send.payload, xfer->send.pay_len);
-       if (xfer->recv.payload != NULL)
+       if (xfer->recv.pay_len > 0)
                kmem_free(xfer->recv.payload, xfer->recv.pay_len);
        cv_destroy(&xfer->cv);
        kmem_free(xfer, sizeof(struct fw_xfer));