Skip to content

lua: support luzer-based testing#14610

Merged
DavidKorczynski merged 1 commit intogoogle:masterfrom
ligurio:ligurio/test-lua-with-luzer
Apr 10, 2026
Merged

lua: support luzer-based testing#14610
DavidKorczynski merged 1 commit intogoogle:masterfrom
ligurio:ligurio/test-lua-with-luzer

Conversation

@ligurio
Copy link
Copy Markdown
Contributor

@ligurio ligurio commented Dec 19, 2025 •
edited
Loading

Depends on #13929
Depends on ligurio/luzer#74
Depends on ligurio/lunapark#179
Depends on ligurio/luzer#82
Related to ligurio/lunapark#180
Related to ligurio/luzer#83

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 19, 2025

ligurio is either the primary contact or is in the CCs list of projects/lua.
ligurio has previously contributed to projects/lua. The previous PR was #12685
ligurio is integrating a new project:
- Main repo: https://github.com/ligurio/luzer
- Criticality score: 0.23736

@ligurio ligurio mentioned this pull request Dec 24, 2025
Open
34 tasks
@ligurio ligurio mentioned this pull request Jan 7, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 6bca742 to 648bcfa Compare January 17, 2026 15:47
@ligurio ligurio marked this pull request as draft January 30, 2026 07:34
ligurio added a commit to ligurio/luzer that referenced this pull request Mar 27, 2026
@ligurio
rockspec: define directories with headers and lib
09a22e4 
Sometimes one need to compile a luzer module with a non-system Lua
library and headers, for which the LUA_LIBRARIES and
LUA_INCLUDE_DIR CMake options are used. The patch adds variables
for the rockspec to set these options via environment variables:
LUA_LIBRARIES and LUA_INCLUDE_DIR.

Needed for google/oss-fuzz#14610
@ligurio ligurio mentioned this pull request Mar 27, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 648bcfa to b14c354 Compare March 27, 2026 12:26
ligurio added a commit to ligurio/lunapark that referenced this pull request Mar 27, 2026
@ligurio
cmake: introduce option ENABLE_LIBFUZZER_STATIC_LINKAGE
10ec8ad 
The patch itnroduces a CMake option ENABLE_LIBFUZZER_STATIC_LINKAGE
that enables static linkage wituh libFuzzer. By default linkage
is disabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 3 times, most recently from 5b5ddaa to ff72ffc Compare March 27, 2026 14:35
@ligurio ligurio marked this pull request as ready for review March 27, 2026 14:40
ligurio added a commit to ligurio/luzer that referenced this pull request Mar 27, 2026
@ligurio
rockspec: define directories with headers and lib
6c4c5ed 
Sometimes one need to compile a luzer module with a non-system Lua
library and headers, for which the LUA_LIBRARIES and
LUA_INCLUDE_DIR CMake options are used. The patch adds variables
for the rockspec to set these options via environment variables:
LUA_LIBRARIES and LUA_INCLUDE_DIR.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 4 times, most recently from 48eff90 to 5aca8b2 Compare March 27, 2026 15:42
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
a752226 
Sometimes one need to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --lua-version 5.1 --verbose --tree=lua_modules \
  luzer-scm-1.rockspec ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
9d68fcd 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --lua-version 5.1 --verbose --tree=lua_modules \
  luzer-scm-1.rockspec ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 6, 2026
@ligurio
rockspec: propagate CMake options
f0be0ac 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers, for which the CMake options
LUA_LIBRARIES and LUA_INCLUDE_DIR are used. The patch adds
variables for the rockspec to set these options via environment
variables: CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR,
ENABLE_LUAJIT and LUAJIT_FRIENDLY_MODE.

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
008c2d7 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT and
LUAJIT_FRIENDLY_MODE:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
96c0f92 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT and
LUAJIT_FRIENDLY_MODE:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
34d1cee 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
CMAKE_LUA_LIBRARIES, CMAKE_LUA_INCLUDE_DIR, ENABLE_LUAJIT,
LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames a CMake variables used for setting custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
8111c89 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
7c4e79f 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
6f9c893 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.
The patch also sorts variables in alphabetical order.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: propagate CMake options
2a58b8a 
Sometimes one needs to build a luzer module using rockspec with
a non-system Lua library and headers or set other CMake options
before building. The patch adds variables for the rockspec to set
the following CMake options via environment variables:
ENABLE_LUAJIT, LUAJIT_FRIENDLY_MODE and OSS_FUZZ:

luarocks install --tree=lua_modules luzer-scm-1.rockspec \
  ENABLE_LUAJIT=ON LUAJIT_FRIENDLY_MODE=ON

The patch also renames CMake variables used for setting a custom
Lua library and Lua include directory: LUA_LIBRARIES to
CMAKE_LUA_LIBRARIES and LUA_INCLUDE_DIR to CMAKE_LUA_INCLUDE_DIR.
This is needed because with empty environment variables
LUA_LIBRARIES and LUA_INCLUDE_DIR passed to luarocks breaks CMake
module for finding Lua package:

CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find Lua (missing: LUA_LIBRARIES LUA_INCLUDE_DIR) (Required is at least version "5.1")
Call Stack (most recent call first):
  /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.28/Modules/FindLua.cmake:236 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:49 (find_package)

After renaming these options can be specified for luarocks.
The patch also sorts variables in alphabetical order.

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce option ENABLE_LIBFUZZER_STATIC_LINKAGE
018efe6 
The patch itnroduces a CMake option ENABLE_LIBFUZZER_STATIC_LINKAGE
that enables static linkage wituh libFuzzer. By default linkage
is disabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio mentioned this pull request Apr 7, 2026
Merged
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: fix propagating variables
927b671 
The previous patch is not set LUA_INCLUDE_DIR when custom Lua
include directory is used. The patch fixes that. Also, rockspec
has missed variables CMAKE_LUA_LIBRARIES and CMAKE_LUA_INCLUDE_DIR.
The patch adds these variables.

Follows up the commit 2a58b8a
("rockspec: propagate CMake options").

Needed for google/oss-fuzz#14610
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce DISABLE_LIBFUZZER_STATIC_LINKAGE
18ac78c 
The patch itnroduces a CMake option DISABLE_LIBFUZZER_STATIC_LINKAGE
that disables static linkage wituh libFuzzer. By default linkage
is enabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 5aca8b2 to 73f555c Compare April 7, 2026 13:40
@ligurio ligurio mentioned this pull request Apr 7, 2026
Merged
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from 73f555c to 0e03ce8 Compare April 7, 2026 14:03
ligurio added a commit to ligurio/luzer that referenced this pull request Apr 7, 2026
@ligurio
rockspec: fix propagating variables
1d91c24 
The previous patch is not set LUA_INCLUDE_DIR when custom Lua
include directory is used. The patch fixes that. Also, rockspec
has missed variables CMAKE_LUA_LIBRARIES and CMAKE_LUA_INCLUDE_DIR.
The patch adds these variables.

Follows up the commit 2a58b8a
("rockspec: propagate CMake options").

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from d24af8f to c169cce Compare April 7, 2026 14:22
ligurio added a commit to ligurio/lunapark that referenced this pull request Apr 7, 2026
@ligurio
cmake: introduce DISABLE_LIBFUZZER_STATIC_LINKAGE
944f891 
The patch itnroduces a CMake option DISABLE_LIBFUZZER_STATIC_LINKAGE
that disables static linkage wituh libFuzzer. By default linkage
is enabled.

Needed for google/oss-fuzz#14610
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch 2 times, most recently from ad3c460 to e4accfb Compare April 7, 2026 14:51
@ligurio
lua: support luzer-based testing
323e75d 
sudo python infra/helper.py build_fuzzers lua
sudo python infra/helper.py check_build lua utf8_char_test

Related to ligurio/lunapark#180
Related to ligurio/luzer#83
@ligurio ligurio force-pushed the ligurio/test-lua-with-luzer branch from e4accfb to 323e75d Compare April 7, 2026 15:25
@ligurio
Copy link
Copy Markdown
Contributor Author

ligurio commented Apr 7, 2026

@DavidKorczynski ok to merge?

@ligurio
Copy link
Copy Markdown
Contributor Author

ligurio commented Apr 9, 2026

@jonathanmetzman ok to merge?

@DavidKorczynski
Copy link
Copy Markdown
Collaborator

DavidKorczynski commented Apr 9, 2026

I haven't followed the luzer PRs closely, but is this a libfuzzer based fuzzing approach? just to confirm it is compatible with the clusterfuzz infra

@ligurio
Copy link
Copy Markdown
Contributor Author

ligurio commented Apr 9, 2026

I haven't followed the luzer PRs closely, but is this a libfuzzer based fuzzing approach? just to confirm it is compatible with the clusterfuzz infra.

Yes, luzer is a libFuzzer-based fuzzing engine. The same tests were already integrated for the Tarantool project, see #14834 and #14656. And the same tests are already in use for LuaJIT testing using ClusterFuzzLite infra.

@DavidKorczynski DavidKorczynski merged commit 8361dd0 into google:master Apr 10, 2026
16 checks passed
@ligurio ligurio deleted the ligurio/test-lua-with-luzer branch April 10, 2026 13:39
ligurio added a commit to ligurio/oss-fuzz that referenced this pull request Apr 11, 2026
@ligurio
lua: build corpus for Lua API tests
aee5cb8 
Follows up the commit 8361dd0
("lua: support luzer-based testing (google#14610)")
ligurio added a commit to ligurio/oss-fuzz that referenced this pull request Apr 11, 2026
@ligurio
lua: build corpus for Lua API tests
f10db68 
Follows up the commit 8361dd0
("lua: support luzer-based testing (google#14610)")
ligurio added a commit to ligurio/oss-fuzz that referenced this pull request Apr 11, 2026
@ligurio
lua: build corpus for Lua API tests
794c6d8 
Follows up the commit 8361dd0
("lua: support luzer-based testing (google#14610)")
ligurio added a commit to ligurio/oss-fuzz that referenced this pull request Apr 11, 2026
@ligurio
lua: build corpus for Lua API tests
c2789f9 
Follows up the commit 8361dd0
("lua: support luzer-based testing (google#14610)")
@ligurio ligurio mentioned this pull request Apr 11, 2026
Merged
DavidKorczynski pushed a commit that referenced this pull request Apr 13, 2026
@ligurio
lua: build corpus for Lua API tests (#15341)
8736907 
Follows up the commit 8361dd0 ("lua:
support luzer-based testing (#14610)")

Follows up #14610
ligurio added a commit to ligurio/oss-fuzz that referenced this pull request Apr 14, 2026
@ligurio
lua: revert back to Ubuntu 20.04
60be614 
Related to google#15359
Related to google#14610
@ligurio ligurio mentioned this pull request Apr 14, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DavidKorczynski DavidKorczynski DavidKorczynski approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ligurio @DavidKorczynski