From: Josh Durgin Date: Wed, 14 Aug 2013 22:50:59 +0000 (-0700) Subject: rados.py: fix Rados() unicode checking X-Git-Tag: v0.68~66^2~7 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=4422f21a6586467a63ce6841552d0f60aa849cf1;p=ceph.git rados.py: fix Rados() unicode checking Check new parameters and check that rados_id is not None again to catch the empty string. Signed-off-by: Josh Durgin Reviewed-by: Sage Weil --- diff --git a/src/pybind/rados.py b/src/pybind/rados.py index 34d83c7b3536..7768f8c39d3f 100644 --- a/src/pybind/rados.py +++ b/src/pybind/rados.py @@ -187,10 +187,14 @@ Rados object in state %s." % (self.state)) self.librados = CDLL('librados.so.2') self.cluster = c_void_p() self.rados_id = rados_id - if rados_id and not isinstance(rados_id, str): + if rados_id is not None and not isinstance(rados_id, str): raise TypeError('rados_id must be a string or None') if conffile is not None and not isinstance(conffile, str): raise TypeError('conffile must be a string or None') + if name is not None and not isinstance(name, str): + raise TypeError('name must be a string or None') + if clustername is not None and not isinstance(clustername, str): + raise TypeError('clustername must be a string or None') if rados_id and name: raise Error("Rados(): can't supply both rados_id and name") if rados_id: diff --git a/src/test/pybind/test_rados.py b/src/test/pybind/test_rados.py index 019a86c2763a..4628a44a652c 100644 --- a/src/test/pybind/test_rados.py +++ b/src/test/pybind/test_rados.py @@ -11,6 +11,16 @@ def test_rados_init_error(): assert_raises(Error, Rados, conffile='', name='invalid') assert_raises(Error, Rados, conffile='', name='bad.invalid') +def test_rados_init_type_error(): + assert_raises(TypeError, Rados, rados_id=u'admin') + assert_raises(TypeError, Rados, rados_id=u'') + assert_raises(TypeError, Rados, name=u'client.admin') + assert_raises(TypeError, Rados, name=u'') + assert_raises(TypeError, Rados, conffile=u'blah') + assert_raises(TypeError, Rados, conffile=u'') + assert_raises(TypeError, Rados, clusternaem=u'blah') + assert_raises(TypeError, Rados, clustername=u'') + def test_rados_init(): with Rados(conffile='', rados_id='admin'): pass