Issue 8603 - [PATCH] adds ldif_open_mem()
Summary: [PATCH] adds ldif_open_mem()
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: libraries (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: 2.5.0
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-20 14:23 UTC by brett@gladserv.com
Modified: 2020-10-14 21:09 UTC (History)
0 users

See Also:


Attachments
brett-sheffield-2017-02-20-ldif_open_mem.patch (1.43 KB, patch)
2020-03-23 02:24 UTC, Quanah Gibson-Mount
Details

Note You need to log in before you can comment on or make changes to this issue.
Description brett@gladserv.com 2017-02-20 14:23:15 UTC
Full_Name: Brett Sheffield
Version: master
OS: Gentoo GNU/Linux
URL: http://www.brettsheffield.com/patches/openldap/brett-sheffield-170220-ldif_open_mem.patch
Submission from: (NULL) (2001:470:6e0d:0:96de:80ff:fe2d:3aa9)


Patch[0] to add ldif_open_mem() function to ldif API.

ldif_open_mem() is the fmemopen(3) equivalent of ldif_open() which opens
an ldif steam from memory, rather than from a file.

I originally added a similar function to my project libgladdb in 2014[1], but it
really belongs in openldap itself as it is generally useful.

The attached modifications to OpenLDAP Software are subject to the following
notice:

Copyright 2014, 2017 Brett Sheffield
Redistribution and use in source and binary forms, with or without modification,
are permitted only as authorized by the OpenLDAP Public License. 

[0] http://www.brettsheffield.com/patches/openldap/brett-sheffield-170220-ldif_open_mem.patch
[1] https://github.com/brettsheffield/gladdb/blob/master/src/ldif.c
Comment 1 Quanah Gibson-Mount 2017-03-17 16:43:03 UTC
moved from Incoming to Software Enhancements
Comment 2 Quanah Gibson-Mount 2020-03-23 02:24:49 UTC
Created attachment 655 [details]
brett-sheffield-2017-02-20-ldif_open_mem.patch
Comment 3 Quanah Gibson-Mount 2020-06-10 15:15:07 UTC
Hi Brett,

Thanks for the contribution.  However it is missing an appropriate assignment of rights statement as noted at https://www.openldap.org/devel/contributing.html#notice

If you can add such a statement to this ITS that would be appreciated.

Regards,
Quanah
Comment 4 brett@gladserv.com 2020-06-10 15:34:27 UTC
Thanks Quanah.

"""
Copyright 2014, 2017 Brett Sheffield
Redistribution and use in source and binary forms, with or without modification,
are permitted only as authorized by the OpenLDAP Public License.

I have not assigned rights and/or interest in this work to any party.
"""

https://www.openldap.org/devel/contributing.html#notice

On 2020-06-10 15:15, openldap-its@openldap.org wrote:
> https://bugs.openldap.org/show_bug.cgi?id=8603
> 
> --- Comment #3 from Quanah Gibson-Mount <quanah@openldap.org> ---
> Hi Brett,
> 
> Thanks for the contribution.  However it is missing an appropriate assignment
> of rights statement as noted at
> https://www.openldap.org/devel/contributing.html#notice
> 
> If you can add such a statement to this ITS that would be appreciated.
> 
> Regards,
> Quanah
> 
> -- 
> You are receiving this mail because:
> You reported the issue.
Comment 5 Ondřej Kuzník 2020-06-11 09:39:22 UTC
Patch is part of the merge request here:
https://git.openldap.org/openldap/openldap/-/merge_requests/80
Comment 6 Quanah Gibson-Mount 2020-06-23 15:06:06 UTC
  • 42d72389 
by Brett Sheffield at 2020-06-23T10:31:08+01:00 
ITS#8603 Add ldif_open_mem()

ldif_open_mem() is the fmemopen(3) equivalent of ldif_open() which opens
an ldif steam from memory, rather than from a file.