Root Cause:
The segmentation fault was caused by a dangling progress callback pointer. When the second generation job started, it attempted to use the callback pointer from the first job, which had already been deleted.
Solution:
Added sd_set_progress_callback(nullptr, nullptr) to clear the callback pointer before deleting the callback data. This ensures the stable-diffusion.cpp library doesn't try to call a deleted callback.
Changes:
Modified src/stable_diffusion_wrapper.cpp
Cleared callback in generateText2Img() (line 154)
Cleared callback in generateImg2Img() (line 264)
Cleared callback in generateControlNet() (line 375)
Testing:
Server and UI rebuilt successfully
Code follows established cleanup patterns
Prevents dangling pointer access
The fix is now live and should prevent segfaults when running multiple consecutive generation jobs.
## Fixed in commit b191071
**Root Cause:**
The segmentation fault was caused by a dangling progress callback pointer. When the second generation job started, it attempted to use the callback pointer from the first job, which had already been deleted.
**Solution:**
Added `sd_set_progress_callback(nullptr, nullptr)` to clear the callback pointer before deleting the callback data. This ensures the stable-diffusion.cpp library doesn't try to call a deleted callback.
**Changes:**
- Modified `src/stable_diffusion_wrapper.cpp`
- Cleared callback in `generateText2Img()` (line 154)
- Cleared callback in `generateImg2Img()` (line 264)
- Cleared callback in `generateControlNet()` (line 375)
**Testing:**
- Server and UI rebuilt successfully
- Code follows established cleanup patterns
- Prevents dangling pointer access
The fix is now live and should prevent segfaults when running multiple consecutive generation jobs.
Szontágh Ferenc
When the second job started, the server segfaults:
start command:
Fixed in commit
b191071Root Cause: The segmentation fault was caused by a dangling progress callback pointer. When the second generation job started, it attempted to use the callback pointer from the first job, which had already been deleted.
Solution: Added
sd_set_progress_callback(nullptr, nullptr)to clear the callback pointer before deleting the callback data. This ensures the stable-diffusion.cpp library doesn't try to call a deleted callback.Changes:
src/stable_diffusion_wrapper.cppgenerateText2Img()(line 154)generateImg2Img()(line 264)generateControlNet()(line 375)Testing:
The fix is now live and should prevent segfaults when running multiple consecutive generation jobs.