b43: silence a bogus gcc warning

=46rom: Frank Lichtenheld <[email protected]>

inititalise ret to 0 to avoid the following bogus warning:
CC [M] drivers/net/wireless/b43/debugfs.o
drivers/net/wireless/b43/debugfs.c: In function =E2=80=98b43_debugfs_re= ad=E2=80=99:
drivers/net/wireless/b43/debugfs.c:355: warning: =E2=80=98ret=E2=80=99 = may be used uninitialized in this function

---
 drivers/net/wireless/b43/debugfs.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

b43/debugfs.c
index 734e70e..d8ac058 100644

@@ -352,7 +352,7 @@ static ssize_t b43_debugfs_read(struct file *file, =

char __user *userbuf,
struct b43_wldev *dev;
struct b43_debugfs_fops *dfops; struct b43_dfs_file *dfile;
- ssize_t ret;
+ ssize_t ret =3D 0;
char *buf;
const size_t bufsize =3D 1024 * 128; const size_t buforder =3D get_order(bufsize);
--=20
1.5.3.4

-------------------------------------------------------

--=20
Greetings Michael.
-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

read=E2=80=99:

may be used uninitialized in this function

Isn't this what uninitialized_var() is for?

--=20
John W. Linville
[email protected]
-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

s_read=E2=80=99:

=99 may be used uninitialized in this function

I'd be OK with that, too.

--=20
Greetings Michael.
-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

gfs_read=E2=80=99:

=99 may be used uninitialized in this function

Erh, I didn't actually knew that existed, sorry.
OTOH in this case it is actually longer to write.

Should I send a changed patch?

Gruesse,
--=20
=46rank Lichtenheld <[email protected]>
www: http://www.djpig.de/
-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

bugfs_read=E2=80=99:

=80=99 may be used uninitialized in this function

Yes.

--=20
Greetings Michael.
-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

use uninitialized_var() to avoid the following bogus warning:
CC [M] drivers/net/wireless/b43/debugfs.o
drivers/net/wireless/b43/debugfs.c: In function =E2=80=98b43_debugfs_re= ad=E2=80=99:
drivers/net/wireless/b43/debugfs.c:355: warning: =E2=80=98ret=E2=80=99 = may be used uninitialized in this function

---
 drivers/net/wireless/b43/debugfs.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Changed to use uninitialized_var()

b43/debugfs.c
index ef0075d..f84d063 100644

@@ -352,7 +352,7 @@ static ssize_t b43_debugfs_read(struct file *file, =

char __user *userbuf,
struct b43_wldev *dev;
struct b43_debugfs_fops *dfops; struct b43_dfs_file *dfile;
- ssize_t ret;
+ ssize_t uninitialized_var(ret);
char *buf;
const size_t bufsize =3D 1024 * 128; const size_t buforder =3D get_order(bufsize);
--=20
1.5.3.4

-
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html