Page MenuHomeFreeBSD

libcuse: Fix memory leak in cuse_init()
Needs ReviewPublic

Authored by zlei on Feb 5 2024, 6:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 27, 4:05 PM
Unknown Object (File)
Tue, Sep 24, 2:28 PM
Unknown Object (File)
Tue, Sep 24, 8:04 AM
Unknown Object (File)
Wed, Sep 18, 12:52 AM
Unknown Object (File)
Sun, Sep 8, 5:58 AM
Unknown Object (File)
Sun, Sep 8, 5:25 AM
Unknown Object (File)
Aug 30 2024, 3:53 PM
Unknown Object (File)
Aug 30 2024, 6:34 AM
Subscribers

Details

Reviewers
kib
markj
Summary

The mutex attributes object should be destroyed after been used.

Where here also check return value of pthread_mutexattr_init() to
prevent potential writing in unallocated memory or NULL pointer
dereferencing when the system is out of memory.

This fix is inspired by the recent commit a03f768612ad
(libstdthreads: destroy mutexattr in mtx_init()) .

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

zlei requested review of this revision.Feb 5 2024, 6:48 AM
lib/libcuse/cuse_lib.c
113

Emm, return value of pthread_mutex_init() is not checked ;(