Bug 20610 - UDM-Passwort in Prozessliste sichtbar
Summary: UDM-Passwort in Prozessliste sichtbar
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: UDM - CLI
Version: UCS 4.4
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 5.2-3-errata
Assignee: Felix Botner
QA Contact: Florian Best
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-05 11:14 CET by Sönke Schwardt-Krummrich
Modified: 2025-10-15 16:35 CEST (History)
11 users (show)

See Also:
What kind of report is it?: Security Issue
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID:
Max CVSS v3 score: 7.3 (CVSS:3.0/AV:N/AC:H/PR:H/UI:R/S:C/C:H/I:N/A:H)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2010-11-05 11:14:05 CET
Derzeit ist das an UDM-CLI übergebene Passwort in der Prozessliste sichtbar, wenn eines angegeben wurde.

$ ps axww | grep udm
/usr/bin/python2.4 /usr/sbin/udm users/user list --binddn cn=qamaster,cn=dc,cn=computers,dc=univention,dc=qa --bindpw F0NNsopR

(Unvollständige) Liste der mögliche Lösungen:
- UDM entfernt Passwort aus Prozessliste (ldapsearch macht dies auch)
- UDM bekommt Passwort per Datei übergeben
Comment 1 Janek Walkenhorst univentionstaff 2013-07-03 17:05:57 CEST
(In reply to Sönke Schwardt-Krummrich from comment #0)
> - UDM entfernt Passwort aus Prozessliste (ldapsearch macht dies auch)
This is not actually a solution, see Bug #20611
Comment 2 Stefan Gohmann univentionstaff 2016-04-25 07:52:05 CEST
This issue has been filed against UCS 2.4.

UCS 2.4 is out of maintenance and many UCS components have vastly changed in
later releases. Thus, this issue is now being closed.

If this issue still occurs in newer UCS versions, please use "Clone this bug".
In this case please provide detailed information on how this issue is affecting
you.
Comment 3 Stefan Gohmann univentionstaff 2019-01-03 07:18:08 CET
This issue has been filled against UCS 4.1. The maintenance with bug and security fixes for UCS 4.1 has ended on 5st of April 2018.

Customers still on UCS 4.1 are encouraged to update to UCS 4.3. Please contact
your partner or Univention for any questions.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or simply reopen the issue. In this case please provide detailed information on how this issue is affecting you.
Comment 4 Felix Botner univentionstaff 2025-04-17 12:02:41 CEST
We have added

  --bindpwdfile

which is used in join script. You don't have to use --bindpw  anymore.
Comment 5 Florian Best univentionstaff 2025-04-17 14:11:54 CEST
(In reply to Felix Botner from comment #4)
> We have added
> 
>   --bindpwdfile
> 
> which is used in join script. You don't have to use --bindpw  anymore.

Yes, but we also have to check, weather we don't use --bindpw(d) anymore in the product.
That's the case for joinscripts.

Leftovers:
> base/univention-system-setup/usr/lib/univention-system-setup/scripts/10_basis/18root_password:  univention-directory-manager users/user modify --binddn "$binddn" --bindpwd "$bindpwd"
> base/univention-system-setup/umc/python/setup/netconf/modules/LdapDns.py:            self.call([self.executable, "--binddn", self.binddn, "--bindpwd", self.bindpwd
> doc/app-center/lifecycle.rst:     --bindpwd "$BINDPWD"

Missing:
* some scripts for the system (re-)configuration.
* appcenter documentation for ISVs

→ Should we just close this or do anything about these leftovers?
Comment 6 Daniel Tröder univentionstaff 2025-04-22 08:41:51 CEST
IMHO, there should be a test in ucslint to ensure that this option is not used anymore in our products.
Comment 7 Felix Botner univentionstaff 2025-04-22 08:50:22 CEST
TODO:
- Fix "leftovers"
- update documentation
- add check to ucslint
Comment 8 Florian Best univentionstaff 2025-04-22 10:06:53 CEST
We already have that ucslint check:
0001-24: Invalid joinscript api …
Comment 10 Johannes Königer univentionstaff 2025-10-09 10:26:54 CEST
Leftovers have been fixed and documentation has been updated.

Packages:

```
Package: univention-system-setup
Version: 15.3.1
Package: univention-directory-manager-rest
Version: 12.3.1
Package: univention-directory-manager-modules
Version: 17.3.3
Package: ucs-test
Version: 12.3.20
```

```
1278cc41d1d3 | chore(univention-system-setup): update system-setup, umc deprecate --bindpwd in favor of --bindpwdfile
8b3d5a2e5f51 | chore(univention-system-setup): update system-setup, umc deprecate --bindpwd in favor of --bindpwdfile
73ce2ff50730 | chore(umc): deprecate --bindpwd in favor of --bindpwdfile
73ce2ff50730 | chore(umc): deprecate --bindpwd in favor of --bindpwdfile
e8f4f6298341 | test(ucs-test): update test, umc deprecate --bindpwd in favor of --bindpwdfile
7685c681c8ea | doc: update doc, umc deprecate --bindpwd in favor of --bindpwdfile
```