3673 - file descriptor leak in registry.

Status: RESOLVED FIXED

(Core :: XPCOM, defect, P4)

Description

[Bruce Mitchener]

Minor .. until someone starts embedding.

FIU: file descriptor 5: "/home/bruce/.mozilla/registry", O_RDWR | O_CREAT
   * File info: -rwx------  1 bruce    employee     32243 Mar 12 09:57
   * File position: 22975
   * This file descriptor was allocated from:
        _open64        [libc.so.1]
        open64         [libthread.so.1]
        nr_OpenFile    [reg.c:373]
        NR_RegOpen     [reg.c:2224]
        nsComponentManagerImpl::PlatformCreateDll(const char*,nsDll**)
[nsComponentManager.cpp:374]
        nsComponentManagerImpl::SyncComponentsInFile(const char*)
[nsComponentManager.cpp:1765]
        nsComponentManagerImpl::SyncComponentsInDir(const char*)
[nsComponentManager.cpp:1692]
        nsComponentManagerImpl::SyncComponentsInPathList(const char*)
[nsComponentManager.cpp:1662]

nsComponentManagerImpl::AutoRegister(nsIComponentManager::RegistrationTime,const
char*) [nsComponentManager.cpp:1581]

nsComponentManager::AutoRegister(nsIComponentManager::RegistrationTime,const
char*) [nsRepository.cpp:157]
        NS_SetupRegistry [nsSetupRegistry.cpp:293]
        NS_SetupRegistry_1 [nsSetupRegistry.cpp:93]
        main           [nsAppRunner.cpp:120]
        _start         [crt1.o]

Comment 1

[Peter Trudelle]

settting p4 for m4

Comment 2

[Scott Collins]

I have no idea why this bug was assigned to me.  I don't have any intimate
knowledge of the registry, or of file descriptors.  I'd be happy to write an
`auto-file' smart pointer to prevent this from happening in the future, but I
suspect the right thing here is just to find and fix the leak.  Re-assigning to
dp so he can fix it or give it to the person who understands it best.

Comment 3

[Suresh Duddi (gone)]

Once we start having a shutdown sequence for xpcom, this should vanish.

Comment 4

[Suresh Duddi (gone)]

We plugged this with XPCOM shutdown.