ldapmodify doesn't report failures (ITS#3057)

[hyc@OpenLDAP.org]

Full_Name: Howard Chu
Version: 2.2.7+, HEAD
OS: Linux
URL: 
Submission from: (NULL) (24.126.120.178)
Submitted by: hyc


The change in rev 1.152 of ldapmodify.c breaks up the synchronous LDAP request
into an async request and a process_response() function. However, error messages
are only printed for failures of the LDAP request function. Failures returned by
the server are silently discarded. It also appears (not sure) that the return
code is not checked, so in an LDIF with multiple modification requests, failures
will not abort processing - the entire file gets processed regardless. The exit
status will reflect the status of the last operation in the file.