zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lordofkey <...@git.apache.org>
Subject [GitHub] zookeeper pull request #586: Zookeeper 3105:Character coding problem occur w...
Date Mon, 06 Aug 2018 04:37:41 GMT
Github user lordofkey commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/586#discussion_r207772138
  
    --- Diff: src/contrib/zkpython/src/c/zookeeper.c ---
    @@ -387,8 +387,8 @@ int parse_acls(struct ACL_vector *acls, PyObject *pyacls)
         PyObject *perms = PyDict_GetItemString( a, "perms" );
     #if PY_MAJOR_VERSION >= 3
         acls->data[i].perms = (int32_t)(PyLong_AsLong(perms));
    -    acls->data[i].id.id = strdup( PyUnicode_AsUnicode( PyDict_GetItemString( a, "id"
) ) );
    -    acls->data[i].id.scheme = strdup( PyUnicode_AsUnicode( PyDict_GetItemString( a,
"scheme" ) ) );
    +    acls->data[i].id.id = strdup( PyBytes_AS_STRING( PyUnicode_AsASCIIString( PyDict_GetItemString(
a, "id" ) ) ) );
    --- End diff --
    
    i thought the value of id can be "anyone" or ip or the string like "usrname:password",
either of them can be represented  as a string. then i tried "anyone", "127.0.0.1/16" and
"usrname:password", and they all worked.


---

Mime
View raw message