From 8334b477a4b0973390b6d59f4cb5aa403511e077 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 19 Jan 2010 10:10:52 -0800 Subject: [PATCH 1/1] dpif-linux: Always set *fnp in make_openvswitch_device(). Some versions of GCC warn about this. Always initializing it seems like the right thing to do, since we "almost always" initialized it before. Reported-by: Neil McKee --- lib/dpif-linux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c index b7ba3062e..e7d43811d 100644 --- a/lib/dpif-linux.c +++ b/lib/dpif-linux.c @@ -571,13 +571,14 @@ make_openvswitch_device(int minor, char **fnp) struct stat s; char fn[128]; + *fnp = NULL; + major = get_openvswitch_major(); if (major < 0) { return -major; } dev = makedev(major, minor); - *fnp = NULL; sprintf(fn, "%s/dp%d", dirname, minor); if (!stat(fn, &s)) { if (!S_ISCHR(s.st_mode)) { -- 2.43.0