Bug 40144

Summary:	search_ext_s() возвращает больше, чем указано в sizelimit=1
Product:	Sisyphus	Reporter:	Andrey Cherepanov <cas>
Component:	samba-dc	Assignee:	Evgeny Sinelnikov <sin>
Status:	NEW ---	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P5	CC:	antohami, cas, cow, darktemplar, evg, george, grenka, imz, lav, mithraen, nbr, nir, qa_viy, rider, sem, shaba, sin, slev, vitty, viy
Version:	unstable
Hardware:	x86_64
OS:	Linux

Description

Andrey Cherepanov 2021-06-02 19:57:57 MSK

# python3
Python 3.8.6 (default, Oct  2 2020, 00:06:33) 
[GCC 9.3.1 20200518 (ALT Sisyphus 9.3.1-alt1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import ldap
>>> con = ldap.initialize('ldaps://10.4.4.163',0)>>> con.bind(who='CN=ivanov,CN=Users,DC=company,DC=localnet',cred='Pa$$word')1
>>> len(con.search_ext_s(base='cn=Users,dc=company,dc=localnet',scope=ldap.SCOPE_SUBTREE, filterstr='(objectClass=posixAccount)',sizelimit=1))
2

В документации (https://www.python-ldap.org/en/python-ldap-3.3.0/reference/ldap.html#):
The amount of search results retrieved can be limited with the sizelimit parameter when using search_ext() or search_ext_s() (client-side search limit). If non-zero not more than sizelimit results are returned by the server.

Comment 1

Andrey Cherepanov 2021-06-02 20:12:10 MSK

LDAPTLS_REQCERT=never python3
...
import ldap
con = ldap.initialize('ldaps://10.4.4.163',0)
con.bind(who='CN=ivanov,CN=Users,DC=company,DC=localnet',cred='Pa$$word')
len(con.search_ext_s(base='cn=Users,dc=company,dc=localnet',scope=ldap.SCOPE_SUBTREE, filterstr='(objectClass=posixAccount)',sizelimit=1))

Актуально и в p9.

Comment 2

Stanislav Levin 2021-06-02 20:42:00 MSK

Какой используется backend(LDAP сервер)?

Comment 3

Stanislav Levin 2021-06-02 21:30:22 MSK

Проверил на 389-ds.
Корректно генерируется исключение ldap.SIZELIMIT_EXCEEDED при превышении sizelimit.

search_ext_s - это обертка над ldap_search_ext из OpenLDAP.

Comment 4

Andrey Cherepanov 2021-06-03 06:30:02 MSK

(Ответ для Stanislav Levin на комментарий #2)
> Какой используется backend(LDAP сервер)?

samba-dc.

Comment 5

Andrey Cherepanov 2021-06-03 08:48:27 MSK

Подтверждаю, что прилетает исключение. Что скажет Евгений Синельников?

Comment 6

Evgeny Sinelnikov 2021-06-03 12:48:20 MSK

(Ответ для Andrey Cherepanov на комментарий #5)
> Подтверждаю, что прилетает исключение. Что скажет Евгений Синельников?

Надо посмотреть. Буду проверять. О результатах отпишусь.

Comment 7

Evgeny Sinelnikov 2021-06-03 13:05:36 MSK

Ну, мы не одиноки:
- https://lists.samba.org/archive/samba/2018-February/213970.html
"I can't find any code enforcing a size limit in Samba's LDAP server."

Samba такая Samba. Будем патчить? ;)

Comment 8

Andrey Cherepanov 2021-06-03 20:35:56 MSK

(Ответ для Evgeny Sinelnikov на комментарий #7)
> Ну, мы не одиноки:
> - https://lists.samba.org/archive/samba/2018-February/213970.html
> "I can't find any code enforcing a size limit in Samba's LDAP server."
> 
> Samba такая Samba. Будем патчить? ;)

Было бы неплохо. И апстриму поможем.

Comment 9

Andrey Cherepanov 2021-06-04 12:49:44 MSK

На samba-dc

Comment 10

Anton Farygin 2026-09-25 14:41:45 MSK

Воспроизводится на Samba 4.19.9, 4.21.9 и 4.22.11: ldapsearch -H ldaps://localhost -x -D administrator -z 1 '(objectClass=user)' → "result: 0 Success", записей 9–17, sizeLimitExceeded нет. Сервер не соблюдает sizelimit клиента — поведение апстрима Samba.