296139 - piping password to ldap CLUs on HP-UX

Status: RESOLVED FIXED

(Directory :: LDAP C SDK, defect)

Description

[Ulf Weltman]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.8) Gecko/20050518 Firefox/1.0.4
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.8) Gecko/20050518 Firefox/1.0.4

When piping a password to the ldap command-line utilities, the tcgetattr() call
I added for bug #281140 fails with ENOTTY when it tries to retrieve the terminal
settings for the stdin file descriptor, because it's not a terminal at that point.

Reproducible: Always

Steps to Reproduce:
1. echo "secret" | ldapsearch -D "cn=testguy,dc=example,dc=com" -s base -b ""
"objectclass=*"


Actual Results:  
tcgetattr: Not a typewriter

Expected Results:  
LDAP result

Comment 1

[Ulf Weltman]

Attachment: Change to not manipulate terminal if stdin is not a tty

Comment 2

[Mark Smith [:mcs]]

Comment on attachment 184968 [details] [diff] [review]
Change to not manipulate terminal if stdin is not a tty

The patch looks good to me.  I should've thought of this problem earlier....

Comment 3

[Rich Megginson]

Checking in common.c;
/cvsroot/mozilla/directory/c-sdk/ldap/clients/tools/common.c,v  <--  common.c
new revision: 5.17; previous revision: 5.16
done