search

lucat / leqm-nrt

An open source implementation of Leq(M) measurement
7 stars 5 forks source link

long path issue with input files, possibly related to previous Bug. #3

Closed jamiegau closed 5 years ago

jamiegau commented 5 years ago

Ok, here is a possibly cause for the CORE DUMP under ubuntu. Looks like a long path issue. Tho I did compile the sndfile lib as suggested.. And it did work. Well I thought it did. But then the problem returned. See this example.

oot@wendy2:/share/FinishingRoom/luca_test_audio_files# leqm-nrt test-6ch-2kHz-20dBFS.wav 
leqm-nrt  Copyright (C) 2011-2013, 2017-2018 Luca Trisciani
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under the GPL v3 licence.
Program will use 1 + 3 slave threads.
Opened file: test-6ch-2kHz-20dBFS.wav
Sample rate: 48000
Channels: 6
Format: 1245187
Frames: 2880000
Leq(M): 84.9929
root@wendy2:/share/FinishingRoom/luca_test_audio_files# leqm-nrt /share/FinishingRoom/luca_test_audio_files/test-6ch-2kHz-20dBFS.wav 
leqm-nrt  Copyright (C) 2011-2013, 2017-2018 Luca Trisciani
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under the GPL v3 licence.
Program will use 1 + 3 slave threads.
*** buffer overflow detected ***: leqm-nrt terminated
Aborted (core dumped)
root@wendy2:/share/FinishingRoom/luca_test_audio_files#

I am pointing it at exactly the same file.. But then I give it the full path to the file.. CORE DUMP.

As such, I have not tried, but it probably will compile with the libsndfile you can install via apt-get.. as when I was doing those tests I was always referencing via full path.

Once you have a quick test of this issue yourself, and likely (trivial) fix, I'll give it a go.

lucat commented 5 years ago

Yes, you are right. Path + filename was limited to 64 characters. This was the problem. Thank you very much. I have extended to 1024 characters. That should be enough.