[issue23875] Incorrect argument parsing in _ssl

2015-04-06 Thread Serhiy Storchaka 

New submission from Serhiy Storchaka:

Argument parsing code for functions _ssl.enum_certificates() and 
_ssl.enum_crls() look not correct. 

if (!PyArg_ParseTupleAndKeywords(args, kwds, s|s:enum_certificates,
 kwlist, store_name)) {
return NULL;
}

The format contains codes for two string arguments. But only one address 
(store_name) is passed. And kwlist contains only one member, store_name.

These functions are provided only on Windows, so I can't check what happen if 
call them with two string arguments. May be crash or memory corruption.

--
components: Extension Modules
messages: 240151
nosy: alex, christian.heimes, dstufft, giampaolo.rodola, janssen, pitrou, 
serhiy.storchaka
priority: normal
severity: normal
status: open
title: Incorrect argument parsing in _ssl
type: behavior
versions: Python 3.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23875
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23875] Incorrect argument parsing in _ssl

2015-04-06 Thread Roundup Robot 

Roundup Robot added the comment:

New changeset e0ba8d3bed7e by Benjamin Peterson in branch '3.4':
remove extra arguments in arg parsing format codes (closes #23875)
https://hg.python.org/cpython/rev/e0ba8d3bed7e

New changeset 5f27e13faae2 by Benjamin Peterson in branch '2.7':
remove extra arguments in arg parsing format codes (closes #23875)
https://hg.python.org/cpython/rev/5f27e13faae2

New changeset c9b9fb86d3fa by Benjamin Peterson in branch 'default':
merge 3.4 (#23875)
https://hg.python.org/cpython/rev/c9b9fb86d3fa

--
nosy: +python-dev
resolution:  - fixed
stage:  - resolved
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23875
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23875] Incorrect argument parsing in _ssl

2015-04-06 Thread Benjamin Peterson 

Benjamin Peterson added the comment:

Good catch.

--
nosy: +benjamin.peterson

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23875
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com