* IA32 (2.6.2-rc1 - 2004-01-23.22.30) - 2 New warnings (gcc 3.2.2)
@ 2004-01-24 11:13 John Cherry
2004-01-24 12:01 ` [PATCH] " Tim Cambrant
0 siblings, 1 reply; 2+ messages in thread
From: John Cherry @ 2004-01-24 11:13 UTC (permalink / raw)
To: linux-kernel
arch/i386/kernel/dmi_scan.c:287: warning: `apm_is_horked_d850md' defined but not used
fs/xfs/xfs_log_recover.c:1534: warning: `flags' might be used uninitialized in this function
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH] Re: IA32 (2.6.2-rc1 - 2004-01-23.22.30) - 2 New warnings (gcc 3.2.2)
2004-01-24 11:13 IA32 (2.6.2-rc1 - 2004-01-23.22.30) - 2 New warnings (gcc 3.2.2) John Cherry
@ 2004-01-24 12:01 ` Tim Cambrant
0 siblings, 0 replies; 2+ messages in thread
From: Tim Cambrant @ 2004-01-24 12:01 UTC (permalink / raw)
To: John Cherry; +Cc: Linux Kernel Mailing List, torvalds
On Sat, Jan 24, 2004 at 03:13:49AM -0800, John Cherry wrote:
> fs/xfs/xfs_log_recover.c:1534: warning: `flags' might be used uninitialized in this function
This patch will initialize the ushort 'flags'. This is _really_ trivial, and
isn't needed, but if it's important to remove as many warnings as possible,
then this patch will sort it out.
--- linux-2.6.2-rc/fs/xfs/xfs_log_recover.c.orig Sat Jan 24 12:49:07 2004
+++ linux-2.6.2-rc/fs/xfs/xfs_log_recover.c Sat Jan 24 12:51:38 2004
@@ -1531,7 +1531,7 @@ xlog_recover_reorder_trans(
xlog_recover_item_t *first_item, *itemq, *itemq_next;
xfs_buf_log_format_t *buf_f;
xfs_buf_log_format_v1_t *obuf_f;
- ushort flags;
+ ushort flags = 0;
first_item = itemq = trans->r_itemq;
trans->r_itemq = NULL;
Apply this if it's really needed. GCC 3.4 doesn't give any warnings though.
Tim Cambrant
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-01-24 12:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-01-24 11:13 IA32 (2.6.2-rc1 - 2004-01-23.22.30) - 2 New warnings (gcc 3.2.2) John Cherry
2004-01-24 12:01 ` [PATCH] " Tim Cambrant
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox