X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=servers%2Fslapd%2Fschema.c;h=e9667799e857a7f37fd0ec04456ac7fe97d9b5b8;hb=dd88fdbcc331dba5e9c7e7346deb3598620125e7;hp=bd1601ce0da03b76c69c08a3afba93f3b84c9248;hpb=dc0eacd40b625258355eea866d62188e5aa7ce3b;p=openldap diff --git a/servers/slapd/schema.c b/servers/slapd/schema.c index bd1601ce0d..e9667799e8 100644 --- a/servers/slapd/schema.c +++ b/servers/slapd/schema.c @@ -2,7 +2,7 @@ /* $OpenLDAP$ */ /* This work is part of OpenLDAP Software . * - * Copyright 1998-2005 The OpenLDAP Foundation. + * Copyright 1998-2009 The OpenLDAP Foundation. * All rights reserved. * * Redistribution and use in source and binary forms, with or without @@ -42,11 +42,11 @@ schema_info( Entry **entry, const char **text ) struct berval vals[5]; struct berval nvals[5]; - e = (Entry *) SLAP_CALLOC( 1, sizeof(Entry) ); + e = entry_alloc(); if( e == NULL ) { /* Out of memory, do something about it */ Debug( LDAP_DEBUG_ANY, - "schema_info: SLAP_CALLOC failed - out of memory.\n", 0, 0, 0 ); + "schema_info: entry_alloc failed - out of memory.\n", 0, 0, 0 ); *text = "out of memory"; return LDAP_OTHER; } @@ -83,7 +83,7 @@ schema_info( Entry **entry, const char **text ) int rc; AttributeDescription *desc = NULL; struct berval rdn = frontendDB->be_schemadn; - vals[0].bv_val = strchr( rdn.bv_val, '=' ); + vals[0].bv_val = ber_bvchr( &rdn, '=' ); if( vals[0].bv_val == NULL ) { *text = "improperly configured subschema subentry"; @@ -102,8 +102,8 @@ schema_info( Entry **entry, const char **text ) return LDAP_OTHER; } - nvals[0].bv_val = strchr( frontendDB->be_schemandn.bv_val, '=' ); - assert( nvals[0].bv_val ); + nvals[0].bv_val = ber_bvchr( &frontendDB->be_schemandn, '=' ); + assert( nvals[0].bv_val != NULL ); nvals[0].bv_val++; nvals[0].bv_len = frontendDB->be_schemandn.bv_len - (nvals[0].bv_val - frontendDB->be_schemandn.bv_val); @@ -117,36 +117,23 @@ schema_info( Entry **entry, const char **text ) } { - struct tm *ltm; -#ifdef HAVE_GMTIME_R - struct tm ltm_buf; -#endif char timebuf[ LDAP_LUTIL_GENTIME_BUFSIZE ]; /* - * According to RFC 2251: + * According to RFC 4512: - Servers SHOULD provide the attributes createTimestamp and - modifyTimestamp in subschema entries, in order to allow clients to - maintain their caches of schema information. + Servers SHOULD maintain the 'creatorsName', 'createTimestamp', + 'modifiersName', and 'modifyTimestamp' attributes for all entries of + the DIT. * to be conservative, we declare schema created * AND modified at server startup time ... */ -#ifdef HAVE_GMTIME_R - ltm = gmtime_r( &starttime, <m_buf ); -#else - ldap_pvt_thread_mutex_lock( &gmtime_mutex ); - ltm = gmtime( &starttime ); -#endif /* HAVE_GMTIME_R */ - lutil_gentime( timebuf, sizeof(timebuf), ltm ); -#ifndef HAVE_GMTIME_R - ldap_pvt_thread_mutex_unlock( &gmtime_mutex ); -#endif - vals[0].bv_val = timebuf; - vals[0].bv_len = strlen( timebuf ); + vals[0].bv_len = sizeof( timebuf ); + + slap_timestamp( &starttime, vals ); if( attr_merge_one( e, ad_createTimestamp, vals, NULL ) ) { /* Out of memory, do something about it */