[devel] [Fwd: [sisyphus] nss ldap bug ( или злой я сегодня)]

[devel] [Fwd: [sisyphus] nss ldap bug ( или злой я сегодня)]

[Aleksey Novodvorsky]

[-- Attachment #1: Type: text/plain, Size: 1 bytes --]



[-- Attachment #2: Type: message/rfc822, Size: 4191 bytes --]

[-- Attachment #2.1.1.1: Type: text/plain, Size: 829 bytes --]

Приветсвую всех!

В nss_ldap-172-alt4 обнаружился препротивнейший баг.
Сегодня полдня потратил на изыскания почему это у меня ls -l стал
завершаться с Segmentation fault. Выяснилось, что в nss_ldap кто-то
нехороший не проверил в одном месте указатель на NULL. В итоге, если
LDAP-сервер по какой-то причине недоступен, то все команды пытающиеся
получить скажем имя пользователя по его uid круто обламываются.
Патч приложен.

(В pam_ldap-125-alt1 тоже прячется злобный SegFault, но на его исправление
сегодня уже не хватило времени. Патчик будет, видимо, завтра)

-- 
Best Regards!           | Когда вам платят за работу, надо по крайней мере
Alexey Voinov           | делать вид, что вы работаете...
                        |                         Б.Виан "Осень в Пекине"
voins@voins.program.ru
voins@online.ru
vns@altlinux.ru


[-- Attachment #2.1.1.2: nss_ldap-patch.patch --]
[-- Type: text/plain, Size: 410 bytes --]

diff -Naurk.orig nss_ldap-172/ldap-nss.c.orig nss_ldap-172/ldap-nss.c
--- nss_ldap-172/ldap-nss.c.orig	Thu Aug 30 07:34:27 2001
+++ nss_ldap-172/ldap-nss.c	Tue Apr  2 14:26:48 2002
@@ -824,7 +824,7 @@
 #ifdef HAVE_LDAP_INITIALIZE
 	}
 #endif
-      if (__session.ls_conn != NULL || cfg->ldc_next == cfg)
+      if (__session.ls_conn != NULL || cfg->ldc_next == cfg || cfg->ldc_next == NULL)
 	{
 	  break;
 	}

[-- Attachment #2.1.2: Type: application/pgp-signature, Size: 232 bytes --]

Re: [devel] [Fwd: [sisyphus] nss ldap bug ( или злой я сегодня)]

[Volkov Serge]

[-- Attachment #1: Type: text/plain, Size: 251 bytes --]

Приветствую

Принято!!!
Сегодня поправлю!!! nss_ldap и жду патча на pam ?



On Wed, 03 Apr 2002 12:08:01 +0400
Aleksey Novodvorsky <aen@altlinux.ru> wrote:

> 
> 


-- 
With best wishes, Volkov Serge		
Network Administrator/Security Administrator 		

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [Fwd: [sisyphus] nss ldap bug ( или злой я сегодня)]

[Alexey Voinov]

[-- Attachment #1.1: Type: text/plain, Size: 383 bytes --]

Volkov Serge wrote
> Приветствую
> 
> Принято!!!
> Сегодня поправлю!!! nss_ldap и жду патча на pam ?
> 
В аттаче.


-- 
Best Regards!           | Когда вам платят за работу, надо по крайней мере
Alexey Voinov           | делать вид, что вы работаете...
                        |                         Б.Виан "Осень в Пекине"
voins@voins.program.ru
voins@online.ru
vns@altlinux.ru


[-- Attachment #1.2: pam_ldap-patch.patch --]
[-- Type: text/plain, Size: 690 bytes --]

diff -Naurk.orig pam_ldap-125/pam_ldap.c.orig pam_ldap-125/pam_ldap.c
--- pam_ldap-125/pam_ldap.c.orig	Mon Sep  3 12:57:26 2001
+++ pam_ldap-125/pam_ldap.c	Wed Apr  3 10:14:58 2002
@@ -820,13 +820,13 @@
 		passwdScope = LDAP_SCOPE_ONELEVEL;
 	      else if (!strcasecmp (s, "base"))
 		passwdScope = LDAP_SCOPE_BASE;
-	    }
-	  s = strchr (s, '?');
-	  if (s != NULL)
-	    {
-	      *s = '\0';
-	      s++;
-	      CHECKPOINTER (passwdFilter = strdup (s));
+	      s = strchr (s, '?');
+	      if (s != NULL)
+	        {
+	          *s = '\0';
+	          s++;
+	          CHECKPOINTER (passwdFilter = strdup (s));
+	        }
 	    }
 	}
       else if (!strcasecmp (k, "ldap_version"))

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

Update: Re: [devel] [Fwd: [sisyphus] nss ldap bug ( или злой я сегодня)]

[Volkov Serge]

[-- Attachment #1: Type: text/plain, Size: 313 bytes --]

Ушли в cvs.lrn:/incoming/Sisyphus/

nss_ldap-172-alt5.src.rpm
pam_ldap-125-alt2.src.rpm

Собственно исправления ранее выявленной ошибки.


On Wed, 03 Apr 2002 12:08:01 +0400
Aleksey Novodvorsky <aen@altlinux.ru> wrote:

> 
> 


-- 
With best wishes, Volkov Serge		
Network Administrator/Security Administrator 		

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]