diff options
author | Gregory P. Smith <greg@krypto.org> | 2018-12-31 01:54:53 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-12-31 01:54:53 (GMT) |
commit | c7e219132aff1e21cb9ccb0a9b570dc6c750039b (patch) | |
tree | a8aa10d5c3c2a904ca9a7065ec618c9326658e86 | |
parent | b474e6774d60fa67d5373e361a0ed53c18b24f53 (diff) | |
download | cpython-c7e219132aff1e21cb9ccb0a9b570dc6c750039b.zip cpython-c7e219132aff1e21cb9ccb0a9b570dc6c750039b.tar.gz cpython-c7e219132aff1e21cb9ccb0a9b570dc6c750039b.tar.bz2 |
Dead code removal from _hashopenssl. (GH-11379)
HASH_OBJ_CONSTRUCTOR has always been defined as 0 since I created
hashlib in Python 2.5. Delete all code associated with it.
-rw-r--r-- | Modules/_hashopenssl.c | 98 | ||||
-rw-r--r-- | Modules/clinic/_hashopenssl.c.h | 42 |
2 files changed, 19 insertions, 121 deletions
diff --git a/Modules/_hashopenssl.c b/Modules/_hashopenssl.c index 6da057a..9091025 100644 --- a/Modules/_hashopenssl.c +++ b/Modules/_hashopenssl.c @@ -28,10 +28,6 @@ #define MUNCH_SIZE INT_MAX -#ifndef HASH_OBJ_CONSTRUCTOR -#define HASH_OBJ_CONSTRUCTOR 0 -#endif - #if (OPENSSL_VERSION_NUMBER < 0x10100000L) || defined(LIBRESSL_VERSION_NUMBER) /* OpenSSL < 1.1.0 */ #define EVP_MD_CTX_new EVP_MD_CTX_create @@ -345,76 +341,23 @@ EVP_repr(EVPobject *self) return PyUnicode_FromFormat("<%U HASH object @ %p>", self->name, self); } -/*[clinic input] -_hashlib.HASH.__init__ as EVP_tp_init - - name as name_obj: object - string as data_obj: object(py_default="b''") = NULL - -A hash is an object used to calculate a checksum of a string of information. - -Methods: - -update() -- updates the current digest with an additional string -digest() -- return the current digest value -hexdigest() -- return the current digest as a string of hexadecimal digits -copy() -- return a copy of the current hash object - -Attributes: - -name -- the hash algorithm being used by this object -digest_size -- number of bytes in this hashes output -[clinic start generated code]*/ - -static int -EVP_tp_init_impl(EVPobject *self, PyObject *name_obj, PyObject *data_obj) -/*[clinic end generated code: output=44766d27757cf851 input=dac22658387f9b5d]*/ -{ - Py_buffer view; - char *nameStr; - const EVP_MD *digest; - - if (data_obj) - GET_BUFFER_VIEW_OR_ERROR(data_obj, &view, return -1); - - if (!PyArg_Parse(name_obj, "s", &nameStr)) { - PyErr_SetString(PyExc_TypeError, "name must be a string"); - if (data_obj) - PyBuffer_Release(&view); - return -1; - } - - digest = EVP_get_digestbyname(nameStr); - if (!digest) { - PyErr_SetString(PyExc_ValueError, "unknown hash function"); - if (data_obj) - PyBuffer_Release(&view); - return -1; - } - if (!EVP_DigestInit(self->ctx, digest)) { - _setException(PyExc_ValueError); - if (data_obj) - PyBuffer_Release(&view); - return -1; - } - - Py_INCREF(name_obj); - Py_XSETREF(self->name, name_obj); - - if (data_obj) { - if (view.len >= HASHLIB_GIL_MINSIZE) { - Py_BEGIN_ALLOW_THREADS - EVP_hash(self, view.buf, view.len); - Py_END_ALLOW_THREADS - } else { - EVP_hash(self, view.buf, view.len); - } - PyBuffer_Release(&view); - } - - return 0; -} - +PyDoc_STRVAR(hashtype_doc, +"HASH(name, string=b\'\')\n" +"--\n" +"\n" +"A hash is an object used to calculate a checksum of a string of information.\n" +"\n" +"Methods:\n" +"\n" +"update() -- updates the current digest with an additional string\n" +"digest() -- return the current digest value\n" +"hexdigest() -- return the current digest as a string of hexadecimal digits\n" +"copy() -- return a copy of the current hash object\n" +"\n" +"Attributes:\n" +"\n" +"name -- the hash algorithm being used by this object\n" +"digest_size -- number of bytes in this hashes output"); static PyTypeObject EVPtype = { PyVarObject_HEAD_INIT(NULL, 0) @@ -438,7 +381,7 @@ static PyTypeObject EVPtype = { 0, /*tp_setattro*/ 0, /*tp_as_buffer*/ Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE, /*tp_flags*/ - EVP_tp_init__doc__, /*tp_doc*/ + hashtype_doc, /*tp_doc*/ 0, /*tp_traverse*/ 0, /*tp_clear*/ 0, /*tp_richcompare*/ @@ -448,16 +391,11 @@ static PyTypeObject EVPtype = { EVP_methods, /* tp_methods */ EVP_members, /* tp_members */ EVP_getseters, /* tp_getset */ -#if 1 0, /* tp_base */ 0, /* tp_dict */ 0, /* tp_descr_get */ 0, /* tp_descr_set */ 0, /* tp_dictoffset */ -#endif -#if HASH_OBJ_CONSTRUCTOR - (initproc)EVP_tp_init, /* tp_init */ -#endif }; static PyObject * diff --git a/Modules/clinic/_hashopenssl.c.h b/Modules/clinic/_hashopenssl.c.h index 5f2b235..43f9651 100644 --- a/Modules/clinic/_hashopenssl.c.h +++ b/Modules/clinic/_hashopenssl.c.h @@ -65,46 +65,6 @@ PyDoc_STRVAR(EVP_update__doc__, #define EVP_UPDATE_METHODDEF \ {"update", (PyCFunction)EVP_update, METH_O, EVP_update__doc__}, -PyDoc_STRVAR(EVP_tp_init__doc__, -"HASH(name, string=b\'\')\n" -"--\n" -"\n" -"A hash is an object used to calculate a checksum of a string of information.\n" -"\n" -"Methods:\n" -"\n" -"update() -- updates the current digest with an additional string\n" -"digest() -- return the current digest value\n" -"hexdigest() -- return the current digest as a string of hexadecimal digits\n" -"copy() -- return a copy of the current hash object\n" -"\n" -"Attributes:\n" -"\n" -"name -- the hash algorithm being used by this object\n" -"digest_size -- number of bytes in this hashes output"); - -static int -EVP_tp_init_impl(EVPobject *self, PyObject *name_obj, PyObject *data_obj); - -static int -EVP_tp_init(PyObject *self, PyObject *args, PyObject *kwargs) -{ - int return_value = -1; - static const char * const _keywords[] = {"name", "string", NULL}; - static _PyArg_Parser _parser = {"O|O:HASH", _keywords, 0}; - PyObject *name_obj; - PyObject *data_obj = NULL; - - if (!_PyArg_ParseTupleAndKeywordsFast(args, kwargs, &_parser, - &name_obj, &data_obj)) { - goto exit; - } - return_value = EVP_tp_init_impl((EVPobject *)self, name_obj, data_obj); - -exit: - return return_value; -} - PyDoc_STRVAR(EVP_new__doc__, "new($module, /, name, string=b\'\')\n" "--\n" @@ -292,4 +252,4 @@ exit: #ifndef _HASHLIB_SCRYPT_METHODDEF #define _HASHLIB_SCRYPT_METHODDEF #endif /* !defined(_HASHLIB_SCRYPT_METHODDEF) */ -/*[clinic end generated code: output=cae09468e2cdbefe input=a9049054013a1b77]*/ +/*[clinic end generated code: output=fe5931d2b301ca12 input=a9049054013a1b77]*/ |