gr-osmosdr setting value in destructor
horiz0n at gmx.net
Sun Jan 10 21:52:05 UTC 2016
Hi Jiri, i'd be happy to accept such patch.
On Sun, 04 Oct 2015 15:58:53 +0200, P <j-pi at seznam.cz> wrote:
> there is in gr-osmosdr a lot of code like:
> hackrf_source_c::~hackrf_source_c ()
> _buf = NULL;
> Setting value in destructor have sense in two case:
> - they are used in some way in destructor of parent class
> - You set then to something like 0xDEADBEEF and when pointer
> from already destroyed class is used it will cause segfault.
> Setting value to NULL is contraproductive because there is a lot of
> checks if value is not NULL, this actually can hide some troubles of
> calling something using already freed pointer.
> I'm suggesting to remove this kind of code. I will prepare patch but now
> I'm waiting if those two pending patches I have send before would be
> accepter or not.
> Is there any opinion which oppose previous claim?
> With best regards,
More information about the osmocom-sdr