The lines valuestr ? valuestr : Py_None, and Py_DECREF(valuestr); in the block

of LDAPmessage_to_python() have triggered a NULL pointer deref warning in clang's static code analyzer. LDAPberval_to_object() can return NULL in case PyBytes_FromStringAndSize() fails. Later the value is dereferenced in Py_DECREF(valuestr);.

The function must verify valuestr. Also Py_BuildValue() can be simplified by using N instead of O. https://docs.python.org/3/c-api/arg.html?highlight=py_buildvalue#c.Py_BuildValue