>Please, look at the Operation allocation procedure in servers/slapd/ >slapi/slapi_ops.c line 144. >o_pb are defined in server/slapd/slap.h as o_hdr->oh_pb. And at that >point o_hdr is NULL, so anyone calling slapi_add_entry_internal, >slapi_search_internal, etc would crash slapd.
This patch looks OK to me. I'm in the middle of testing a bunch of other SLAPI changes, so I will commit this once I have it tested too. -- Luke --