Re: [Fedora-directory-devel] Please Review: (251262) ldap-agent needs to find slapd.stats file dynamically

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

 



On Wed, 2007-08-08 at 20:25 -0600, Richard Megginson wrote:
> Andrew Bartlett wrote:
> > On Tue, 2007-08-07 at 23:26 -0700, Nathan Kinder wrote:
> >   
> >> Rich brought up a good point in the bug report about the need to handle
> >> wrapped lines in the dse.ldif.
> >>
> >> This new set of diffs addresses this concern by using the ldif parsing
> >> functions from the libldif library that is part of the Mozilla LDAP C 
> >> SDK.  I
> >> also did some error checking cleanup, particularly ensuring that we 
> >> close any
> >> files that we opened if we encounter an error condition while processing the
> >> config.
> >>
> >> I have also tested that these changes work when there is a wrapped line for
> >> one of the attributes that we read in dse.ldif.
> >>
> >> https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=160880&action=diff
> >>     
> >
> > This may well fix my bug
> > https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239754
> >   
> I don't think so.  I think the crash you reported was in slapd?  
> Nathan's fix is for the snmp subagent.

Sure, but the issue starts because we could feed in invalid ldif in the
dse.ldif.  

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Red Hat Inc.                  http://redhat.com

Attachment: signature.asc
Description: This is a digitally signed message part

--
Fedora-directory-devel mailing list
Fedora-directory-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-directory-devel

[Index of Archives]     [Fedora Directory Announce]     [Fedora Users]     [Older Fedora Users Mail]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Review]     [Fedora Art]     [Fedora Music]     [Fedora Packaging]     [CentOS]     [Fedora SELinux]     [Big List of Linux Books]     [KDE Users]     [Fedora Art]     [Fedora Docs]

  Powered by Linux