ASan reports stack-buffer-underflow about aerospike-server

[kportertx]

CF_NEVER_FAILS would have terminated execution if that read had failed. It appears that the file we are trying to read here contains 0 bytes and the calling function assumes that this wouldn't happen.

The file that was being read follows this pattern "/sys/devices/system/cpu/cpu%hu/node%hu/cpu%hu/topology/core_id". Would you be able to verify that there are empty files that match this pattern on your system?

[zouyonghao]

Hi @kportertx , thanks for your responding.
I'm not sure about that, I'm using a fault injection tool to test this project actually.
But I think it makes sense to fix this issue.

[kportertx]

Which part are you not sure about?

Do you think it could be the fault injection that caused one more of these files to be empty in your environment?

[zouyonghao]

Do you think it could be the fault injection that caused one more of these files to be empty in your environment?

Yes.

[akshatakanihal]

@zouyonghao I would like to work on this issue. could you please give me permission to work on this issue?

[zouyonghao]

Sorry @akshatakanihal , I do not have that permission.

[akshatakanihal]

Can I work on the issue and submit for merge directly?. Is it compulsory to take permission to work on this issue?. I asked permission to show it as a proof to show it to my professor that I m working on it. Thank you

…

On Sat, 10 Sept 2022, 10:59 am Yong-Hao Zou, ***@***.***> wrote: Sorry @akshatakanihal <https://github.com/akshatakanihal> , I do not have that permission. — Reply to this email directly, view it on GitHub <#33 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGFA5D5WRSUF4A2WTXRY3YLV5QMCHANCNFSM5HF3WRNA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[kportertx]

I think we may have taken care of this and had forgotten to update this ticket. Technically it couldn't happen where the utility functions are used, but, iirc, the utilities are written to be generic and would be incorrect if used generically.

I'll need to verify though.