Call to build_initial_monmap() is one stone two birds.  Explicitly it
initializes err variable. Implicitly it initializes ->monmap via call to
kzalloc().  We should only declare err and ->monmap is taken care of by
ceph_monc_init() prototype.
Signed-off-by: Jean Sacren <sakiwit@gmail.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
 
 int ceph_monc_init(struct ceph_mon_client *monc, struct ceph_client *cl)
 {
-       int err = 0;
+       int err;
 
        dout("init\n");
        memset(monc, 0, sizeof(*monc));
        monc->client = cl;
-       monc->monmap = NULL;
        mutex_init(&monc->mutex);
 
        err = build_initial_monmap(monc);