Possible Bug on Android when Setting Period Size

[cdigit]

There's a possible bug that was also reported in Oboe:
Several days ago I picked up a used Samsung Galaxy A10s for testing. It generated an error when I set the period size: maybeConvertDeviceData() conversion size 1024 too large for buffer 512

I apologize for the sketchy details that follow, but It was a hectic time when all of this was going on. The logs are below. I didn't get a chance to capture pertinent info around the android version and API levels because the phone did an auto update. I think it was Android 10 and I think I had a simple duplex example running.

I tried increasing the period size from 512 to 1024 and got the following error, which is double the buffer size of the original error:
Abort message: 'maybeConvertDeviceData() conversion size 2048 too large for buffer 1024'

I search around and saw that people were complaining about the error in Oboe. The Oboe team did something to fix it (something about using a proxy for callback size adapter): Crash when setFramesPerCallback(#frames) is used for input stream #778

When my phone updated the android version, this problem went away.

I wouldn't mind factory resetting the phone to try to get the exact versions to reproduce it, if necessary.
I just realized that there is a build fingerprint in the logs for the phone, when I search it, I get this version info: https://samfw.com/firmware/SM-A107F/MID/A107FXXU8BUF1

D/loglevel: flutter( 4876): start CAlled
D/miniaudio( 4876): DEBUG: Failed to initialize WASAPI backend.
D/miniaudio( 4876): DEBUG: Failed to initialize DirectSound backend.
D/miniaudio( 4876): DEBUG: Failed to initialize WinMM backend.
D/miniaudio( 4876): DEBUG: Failed to initialize Core Audio backend.
D/miniaudio( 4876): DEBUG: Failed to initialize sndio backend.
D/miniaudio( 4876): DEBUG: Failed to initialize audio(4) backend.
D/miniaudio( 4876): DEBUG: Failed to initialize OSS backend.
D/miniaudio( 4876): DEBUG: Failed to initialize PulseAudio backend.
D/miniaudio( 4876): DEBUG: Failed to initialize ALSA backend.
D/miniaudio( 4876): DEBUG: Failed to initialize JACK backend.
D/miniaudio( 4876): DEBUG: Attempting to initialize AAudio backend...
D/miniaudio( 4876): DEBUG: Loading library: libaaudio.so
D/miniaudio( 4876): DEBUG: Loading symbol: AAudio_createStreamBuilder
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_delete
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setDeviceId
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setDirection
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setSharingMode
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setFormat
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setChannelCount
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setSampleRate
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setBufferCapacityInFrames
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setFramesPerDataCallback
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setDataCallback
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setErrorCallback
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setPerformanceMode
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setUsage
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setContentType
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_setInputPreset
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStreamBuilder_openStream
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_close
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getState
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_waitForStateChange
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getFormat
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getChannelCount
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getSampleRate
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getBufferCapacityInFrames
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getFramesPerDataCallback
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_getFramesPerBurst
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_requestStart
D/miniaudio( 4876): DEBUG: Loading symbol: AAudioStream_requestStop
D/miniaudio( 4876): DEBUG: System Architecture:
D/miniaudio( 4876): DEBUG:   Endian: LE
D/miniaudio( 4876): DEBUG:   SSE2:   NO
D/miniaudio( 4876): DEBUG:   AVX2:   NO
D/miniaudio( 4876): DEBUG:   NEON:   YES
I/AAudio  ( 4876): AAudioStreamBuilder_openStream() called ----------------------------------------
I/AudioStreamBuilder( 4876): rate   =      0, channels  = 2, format   = 5, sharing = EX, dir = INPUT
I/AudioStreamBuilder( 4876): device =      0, sessionId = -1, perfMode = 12, callback: ON with frames = 512
I/AudioStreamBuilder( 4876): usage  =      0, contentType = 0, inputPreset = 0, allowedCapturePolicy = 0
D/AudioStreamBuilder( 4876): build() EXCLUSIVE sharing mode not supported. Use SHARED.
D/        ( 4876): PlayerBase::PlayerBase()
I/AudioRecord( 4876): createRecord_l(0): AUDIO_INPUT_FLAG_FAST successful; frameCount 1536 -> 4096
W/AudioRecord( 4876): createRecord_l(116): Server adjusted notificationFrames from 512 to 240 for frameCount 4096
W/AudioStreamRecord( 4876): open() sampleRate changed from 0 to 48000
D/AudioStreamRecord( 4876): open() setup I16>FLOAT conversion buffer with 512 frames
W/AudioStreamRecord( 4876): open() flags changed from 0x00000005 to 0x00000001
I/AAudio  ( 4876): AAudioStreamBuilder_openStream() returns 0 = AAUDIO_OK for s#1 ----------------
I/AAudio  ( 4876): AAudioStreamBuilder_openStream() called ----------------------------------------
I/AudioStreamBuilder( 4876): rate   =      0, channels  = 2, format   = 5, sharing = EX, dir = OUTPUT
I/AudioStreamBuilder( 4876): device =      0, sessionId = -1, perfMode = 12, callback: ON with frames = 512
I/AudioStreamBuilder( 4876): usage  =      0, contentType = 0, inputPreset = 0, allowedCapturePolicy = 0
D/AudioStreamBuilder( 4876): build() EXCLUSIVE sharing mode not supported. Use SHARED.
D/        ( 4876): PlayerBase::PlayerBase()
D/AudioStreamTrack( 4876): open(), request notificationFrames = 512, frameCount = 1536
I/AudioTrack( 4876): createTrack_l(-1390921589): AUDIO_OUTPUT_FLAG_FAST successful; frameCount 1536 -> 1536
D/AudioTrack( 4876): setVolume(1.000000, 1.000000) pid : 4876
W/AudioStreamTrack( 4876): open() sampleRate changed from 0 to 48000
W/AudioStreamTrack( 4876): open() flags changed from 0x00000104 to 0x00000004
I/AAudio  ( 4876): AAudioStreamBuilder_openStream() returns 0 = AAUDIO_OK for s#2 ----------------
D/miniaudio( 4876): INFO: [AAudio]
D/miniaudio( 4876): INFO:   Default Playback Device (Capture)
D/miniaudio( 4876): INFO:     Format:      32-bit IEEE Floating Point -> 32-bit IEEE Floating Point
D/miniaudio( 4876): INFO:     Channels:    2 -> 2
D/miniaudio( 4876): INFO:     Sample Rate: 48000 -> 48000
D/miniaudio( 4876): INFO:     Buffer Size: 512*8 (4096)
D/miniaudio( 4876): INFO:     Conversion:
D/miniaudio( 4876): INFO:       Pre Format Conversion:  NO
D/miniaudio( 4876): INFO:       Post Format Conversion: NO
D/miniaudio( 4876): INFO:       Channel Routing:        NO
D/miniaudio( 4876): INFO:       Resampling:             NO
D/miniaudio( 4876): INFO:       Passthrough:            YES
D/miniaudio( 4876): INFO:       Channel Map In:         {CHANNEL_FRONT_LEFT CHANNEL_FRONT_RIGHT}
D/miniaudio( 4876): INFO:       Channel Map Out:        {CHANNEL_FRONT_LEFT CHANNEL_FRONT_RIGHT}
D/miniaudio( 4876): INFO:   Default Capture Device (Playback)
D/miniaudio( 4876): INFO:     Format:      32-bit IEEE Floating Point -> 32-bit IEEE Floating Point
D/miniaudio( 4876): INFO:     Channels:    2 -> 2
D/miniaudio( 4876): INFO:     Sample Rate: 48000 -> 48000
D/miniaudio( 4876): INFO:     Buffer Size: 512*3 (1536)
D/miniaudio( 4876): INFO:     Conversion:
D/miniaudio( 4876): INFO:       Pre Format Conversion:  NO
D/miniaudio( 4876): INFO:       Post Format Conversion: NO
D/miniaudio( 4876): INFO:       Channel Routing:        NO
D/miniaudio( 4876): INFO:       Resampling:             NO
D/miniaudio( 4876): INFO:       Passthrough:            YES
D/miniaudio( 4876): INFO:       Channel Map In:         {CHANNEL_FRONT_LEFT CHANNEL_FRONT_RIGHT}
D/miniaudio( 4876): INFO:       Channel Map Out:        {CHANNEL_FRONT_LEFT CHANNEL_FRONT_RIGHT}
D/AAudio  ( 4876): AAudioStream_requestStart(s#1) called --------------
D/        ( 4876): PlayerBase::start() from IPlayer
D/AudioStreamLegacy( 4876): onAudioDeviceUpdate() devId 60 => 60
D/AAudio  ( 4876): AAudioStream_requestStart(s#1) returned 0 ---------
D/AudioStreamLegacy( 4876): onAudioDeviceUpdate() devId 60 => 60
D/AAudio  ( 4876): AAudioStream_requestStart(s#2) called --------------
D/        ( 4876): PlayerBase::start() from IPlayer
D/AudioStreamLegacy( 4876): onAudioDeviceUpdate() devId 59 => 59
D/AAudio  ( 4876): AAudioStream_requestStart(s#2) returned 0 ---------
F/AudioStreamRecord( 4876): maybeConvertDeviceData() conversion size 1024 too large for buffer 512
F/libc    ( 4876): Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 7071 (AudioRecord), pid 4876 (m.example.c_ffi)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'samsung/a10sxx/a10s:10/QP1A.190711.020/A107FXXU8BUF1:user/release-keys'
Revision: '0'
ABI: 'arm'
Timestamp: 2023-01-25 14:03:00-0500
pid: 4876, tid: 7071, name: AudioRecord  >>> com.example.c_ffi <<<
uid: 10199
signal 6 (SIGABRT), code -1 (SI_QUEUE), fault addr --------
Abort message: 'maybeConvertDeviceData() conversion size 1024 too large for buffer 512'
    r0  00000000  r1  00001b9f  r2  00000006  r3  addfbbd0
    r4  addfbbe4  r5  addfbbc8  r6  0000130c  r7  0000016b
    r8  addfbbe0  r9  addfbbd0  r10 addfbc00  r11 addfbbf0
    ip  00001b9f  sp  addfbba0  lr  ed4002c3  pc  ed4002d6
backtrace:
      #00 pc 0005f2d6  /apex/com.android.runtime/lib/bionic/libc.so (abort+166) (BuildId: d84af0b640b3e9d365f37fe8072558b3)
      #01 pc 00005a31  /system/lib/liblog.so (__android_log_assert+176) (BuildId: aa584ee968fc512853ed4a7a5ccbd5ec)
      #02 pc 0001580b  /system/lib/libaaudio.so (aaudio::AudioStreamRecord::maybeConvertDeviceData(void const*, int)+58) (BuildId: ffb14d173b3db690330611f57356e393)
      #03 pc 00014d31  /system/lib/libaaudio.so (aaudio::AudioStreamLegacy::onProcessFixedBlock(unsigned char*, int)+100) (BuildId: ffb14d173b3db690330611f57356e393)
      #04 pc 00016d95  /system/lib/libaaudio.so (FixedBlockWriter::processVariableBlock(unsigned char*, int)+66) (BuildId: ffb14d173b3db690330611f57356e393)
      #05 pc 00014ef5  /system/lib/libaaudio.so (aaudio::AudioStreamLegacy::processCallbackCommon(int, void*)+392) (BuildId: ffb14d173b3db690330611f57356e393)
      #06 pc 00034f1f  /system/lib/libaudioclient.so (android::AudioRecord::processAudioBuffer()+838) (BuildId: 6c6771e46f224f4fa40446edd4014ab1)
      #07 pc 0003584b  /system/lib/libaudioclient.so (android::AudioRecord::AudioRecordThread::threadLoop()+122) (BuildId: 6c6771e46f224f4fa40446edd4014ab1)
      #08 pc 0000d937  /system/lib/libutils.so (android::Thread::_threadLoop(void*)+318) (BuildId: b7e8802c3a57e97e1f603225f6e6c6aa)
      #09 pc 000a5c21  /system/lib/libandroid_runtime.so (android::AndroidRuntime::javaThreadShell(void*)+84) (BuildId: b3379d2dd827b9e5f7082879f79f0bb1)
      #10 pc 000a60d3  /apex/com.android.runtime/lib/bionic/libc.so (__pthread_start(void*)+20) (BuildId: d84af0b640b3e9d365f37fe8072558b3)
      #11 pc 00060783  /apex/com.android.runtime/lib/bionic/libc.so (__start_thread+30) (BuildId: d84af0b640b3e9d365f37fe8072558b3)
Lost connection to device.
Exited

[mackron]

Thanks. I don't have a way to reproduce this. I've pushed an experimental workaround to the dev branch. Are you able to give that a try? You'll need to add this to your device config or else the workaround won't be used:

deviceConfig.aaudio.enableCompatibilityWorkarounds = MA_TRUE;

This change will simply not set the data callback size. I think that's what Oboe is doing to work around it.

[cdigit]

Will try this and report back.