[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Phpgroupware-tracker] [support #100802] NIS auth and autoaccounts fail

From: Dr. Christian Böttger
Subject: [Phpgroupware-tracker] [support #100802] NIS auth and autoaccounts fail (patch)
Date: Fri, 26 Mar 2004 16:30:56 -0500
User-agent: Mozilla/5.0 (X11; U; Linux i686; de-AT; rv:1.6) Gecko/20040113

This mail is an automated notification from the support tracker
 of the project: phpGroupWare.

[support #100802] Latest Modifications:

Changes by: 
                Dr. Christian Böttger <address@hidden>
                Fri 03/26/04 at 21:30 (Europe/Berlin)

            What     | Removed                   | Added
         Assigned to | None                      | skwashd

[support #100802] Full Item Snapshot:

URL: <>
Project: phpGroupWare
Submitted by: Nils Rennebarth
On: Wed 04/24/02 at 13:48

Category:  Feature Request
Priority:  5 - Normal
Severity:  3 - Ordinary
Resolution:  None
Assigned to:  skwashd
Originator Email:  
Status:  Open

Summary:  NIS auth and autoaccounts fail (patch)

Original Submission:  
When using "Auto create account records for authenticated
users"=Yes, a user that successfully authenticated
will get an error message and a halted session.

The reason is that


in line 325 of phpgwapi/inc/
is called before checking if account_lid exists and
accounts->get_type isn't ready for nonexistant accounts.

Either repair get_type to return "" if the account is
inexistant or check for existence first.

The following patch implements the first strategy:

--- phpgwapi/inc/      
 Tue Apr 23 17:38:54 2002
+++ phpgwapi/inc/     Wed Apr
24 14:48:14 2002
@@ -226,4 +226,8 @@
                        $account_id =
+                       if($account_id == "")
+                       {
+                               return "";
+                       }
if(@isset($account_type[$account_id]) &&

Follow-up Comments

Date: Mon 12/09/02 at 00:19         By: skwashd

This is not really a feature request, but the bugs category
has been removed so it can stay in the cat :)

For detailed info, follow this link:

  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]