From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Wed, 17 Jul 2013 08:53:56 +0100 Subject: [Cluster-devel] [gfs2-utils PATCH 1/7] fsck.gfs2: Fix reference to uninitialized variable In-Reply-To: <51E62BF2.5070007@redhat.com> References: <08b2ef646a21c19d0cdd1212fe8d4b4a8d7ee2b2.1373979296.git.rpeterso@redhat.com> <51E62BF2.5070007@redhat.com> Message-ID: <1374047636.2707.7.camel@menhir> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi, On Wed, 2013-07-17 at 07:30 +0200, Fabio M. Di Nitto wrote: > On 07/16/2013 02:56 PM, Bob Peterson wrote: > > This patch initializes a variable so that it no longer references > > it uninitialized. > > > > rhbz#984085 > > --- > > gfs2/fsck/initialize.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/gfs2/fsck/initialize.c b/gfs2/fsck/initialize.c > > index b01b240..936fd5e 100644 > > --- a/gfs2/fsck/initialize.c > > +++ b/gfs2/fsck/initialize.c > > @@ -832,7 +832,7 @@ static int get_lockproto_table(struct gfs2_sbd *sdp) > > { > > FILE *fp; > > char line[PATH_MAX]; > > - char *cluname, *end; > > + char *cluname, *end = NULL; > > const char *fsname, *cfgfile = "/etc/cluster/cluster.conf"; > > Just spotted this reference to cluster.conf ^^ remember it doesn't exist > anymore in the new era. > > Fabio > Yes, agreed. It looks like this is trying to repair the superblock, but I don't think that it is a good idea to assume that lack of cluster.conf means that the fs is not clustered. Nor should be be trying to use the lock table name from cluster.conf since we might be repairing a filesystem on a different cluster to the one on which it was mounted and supposed to work, so it might just result in putting incorrect information in to this field. Perhaps better to set it to lock_dlm with a cluster name of unknown and then prompt the user to use gfs2tune to reset the cluster name afterwards, or something like that, Steve.