On Wednesday 13 May 2015 14:11:43 J. Bruce Fields wrote: > > OK, I guess. How about simplifying slightly and doing it this way?--b > > commit 3ae81ac291ec > Author: Arnd Bergmann <arnd@xxxxxxxx> > Date: Tue May 12 23:31:29 2015 +0200 > > nfsd: work around a gcc-5.1 warning > > gcc-5.0 warns about a potential uninitialized variable use in nfsd: > > fs/nfsd/nfs4state.c: In function 'nfsd4_process_open2': > fs/nfsd/nfs4state.c:3781:3: warning: 'old_deny_bmap' may be used uninitialized in this function [-Wmaybe-uninitialized] > reset_union_bmap_deny(old_deny_bmap, stp); > ^ > fs/nfsd/nfs4state.c:3760:16: note: 'old_deny_bmap' was declared here > unsigned char old_deny_bmap; > ^ > > This is a false positive, the code path that is warned about cannot > actually be reached. > > This adds an initialization for the variable to make the warning go > away. > > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxx> > > Works for me, thanks! Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html