Skip to content

Commit 874c68a

Browse files
hardening: implement sql_buffer_t, refactor bulk queries, and add test suite (#466)
Rebased onto develop after #514 merge. Resolved 13 conflicts in poller.c by preserving the conditional output_regex detection (regex_col) from #514 instead of hardcoded column references. Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
1 parent 70459ac commit 874c68a

21 files changed

Lines changed: 2775 additions & 585 deletions

.github/warning-baseline.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
0

.gitignore

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,9 @@
1919
# | http://www.cacti.net/ |
2020
# +-------------------------------------------------------------------------+
2121

22+
# Build Artifacts
2223
.deps
24+
.libs
2325
Makefile
2426
Makefile.in
2527
configure
@@ -28,5 +30,22 @@ autom4te.cache/
2830
config/
2931
libtool
3032
/spine
33+
/spine-conf
3134
*.o
35+
*.lo
36+
*.la
3237
*.m4
38+
.dirstamp
39+
40+
# Project/IDE specific patterns and local worktrees to ensure repository hygiene
41+
.worktrees/
42+
.omc/
43+
.DS_Store
44+
*.swp
45+
*.log
46+
*~
47+
# Sanitizer logs
48+
asan.*
49+
ubsan.*
50+
tsan.*
51+
valgrind.*

Makefile.am

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,14 @@ man_MANS = spine.1
3333

3434
EXTRA_DIST = spine.1 uthash.h
3535

36-
# Docker targets — require Dockerfile and Dockerfile.dev (from PR #401)
36+
# Unit tests (standalone, no spine build required)
37+
TESTS = tests/unit/test_sql_buffer
38+
check_PROGRAMS = tests/unit/test_sql_buffer
39+
tests_unit_test_sql_buffer_SOURCES = tests/unit/test_sql_buffer.c
40+
tests_unit_test_sql_buffer_CFLAGS = -I$(top_srcdir) $(CMOCKA_CFLAGS)
41+
tests_unit_test_sql_buffer_LDADD = $(CMOCKA_LIBS)
42+
43+
# Docker targets
3744
.PHONY: docker docker-dev verify cppcheck
3845

3946
docker:

0 commit comments

Comments
 (0)