Project

General

Profile

Actions

Bug #12801

closed

libdiskmgt leaks PROM device information handles like a sieve

Added by Joshua M. Clulow about 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
lib - userland libraries
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

In the findevs() routine, we call di_prom_init() to get a PROM device information handle. This handle is stored in the struct search_args, but does not appear to be cleaned up afterwards.

For potentially flimsy reasons, /dev/openprom allows at most 32 handles to be opened at once; a persistent process which uses libdiskmgmt without exiting can end up hanging on to a lot of these handles. While it's conceivable that /dev/openprom should be more flexible, it's assuredly not correct to leak these handles.


Related issues

Related to illumos gate - Bug #8709: teach libdiskmgt about nvme, sata, and xenClosedYuri Pankov2017-10-07

Actions
Actions

Also available in: Atom PDF