Resolves: bug 457846
    Bug Description: The Windows Sync API should have plug-in points - part 3
    Reviewed by: nkinder (Thanks!)
    Fix Description: It turns out I was a little bit too aggressive in removing memory leaks, and broke outbound modify processing.  I should not have freed new_dn since it is used elsewhere.  There was an earlier memory leak related to the way new_dn was initialized, but that was fixed elsewhere.  The real fix is this:
    -               slapi_sdn_free(&new_dn);
    The other fixes are lots of log messages I added to help debug this problem.
    Platforms tested: RHEL5
    Flag Day: no
    Doc impact: yes - plugin guide