search

MarvellEmbeddedProcessors / atf-marvell

BSD 3-Clause "New" or "Revised" License
23 stars 24 forks source link

Leave space for trailing \0 after MAX_FILENAME #9

Open berolinux opened 6 years ago

berolinux commented 6 years ago

Detected by gcc 8.0.1 warnings

Signed-off-by: Bernhard Rosenkränzer bero@lindev.ch

afaerber commented 6 years ago

Ping?

Reviewed-by: Andreas Färber afaerber@suse.de

Also needed for openSUSE Tumbleweed with GCC 8.1.1.

kostapr commented 6 years ago

I will take it to our git for the next drop. Cannot merge with the existing release here.

Kosta

On Sun, Jun 10, 2018, 20:16 Andreas Färber notifications@github.com wrote:

Ping?

Reviewed-by: Andreas Färber afaerber@suse.de

Also needed for openSUSE Tumbleweed with GCC 8.1.1.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/MarvellEmbeddedProcessors/atf-marvell/pull/9#issuecomment-396065705, or mute the thread https://github.com/notifications/unsubscribe-auth/AIhiYj8dypTiBepJzsnZpPuEs0ewnwS9ks5t7VRRgaJpZM4TvVWx .

kostapr commented 6 years ago

added couple of similar fixes and pushed to internal ATF 1.5 branch for the next release. Will push it into ATF mainline once our SoC support sources are accepted there.

afaerber commented 6 years ago

This fix is still missing in the new 18.06 branches - needed a slight rebase.