[ovs-dev] [PATCH 3/3] ovs-wdt: Mark variable static and use NULL for a null pointer.
Jesse Gross
jesse at nicira.com
Thu Feb 11 16:45:50 UTC 2010
This whole set looks good. Thanks.
On Wed, Feb 10, 2010 at 7:55 PM, Ben Pfaff <blp at nicira.com> wrote:
> Neither is really a bug but it's cleaner this way.
>
> Found by sparse (http://sparse.wiki.kernel.org/).
> ---
> utilities/ovs-wdt.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/utilities/ovs-wdt.c b/utilities/ovs-wdt.c
> index fc3efb1..fedc65c 100644
> --- a/utilities/ovs-wdt.c
> +++ b/utilities/ovs-wdt.c
> @@ -1,4 +1,4 @@
> -/* Copyright (c) 2008, 2009 Nicira Networks, Inc.
> +/* Copyright (c) 2008, 2009, 2010 Nicira Networks, Inc.
> *
> * Licensed under the Apache License, Version 2.0 (the "License");
> * you may not use this file except in compliance with the License.
> @@ -30,7 +30,7 @@
> #define DEFAULT_INTERVAL 1
> #define DEFAULT_TIMEOUT 30
>
> -int fd = -1;
> +static int fd = -1;
>
> /* The WDT is automatically enabled when /dev/watchdog is opened. If we
> * do not send the magic value to the device first before exiting, the
> @@ -134,7 +134,7 @@ int main(int argc, char *argv[])
> {"help", no_argument, NULL, 'h'},
> {"verbose", no_argument, NULL, 'v'},
> {"version", no_argument, NULL, 'V'},
> - {0, 0, 0, 0}
> + {NULL, 0, NULL, 0}
> };
>
> setup_signal();
> --
> 1.6.6.1
>
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20100211/c8aec6e3/attachment-0003.html>
More information about the dev
mailing list