https://gitlab.synchro.net/main/sbbs/-/commit/d63dd43c9b9a1cc24b268829
Modified Files:
src/uifc/uifc32.c
Log Message:
Prevent dereferencing NULL bar or cur with WIN_DYN

This would be an API violation, but we shouldn't crash about it.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/935afe75ac88dd8094e58e71
Modified Files:
src/uifc/uifc32.c
Log Message:
Fix buffer overflow when upop() is called with a string > 73 chars

result in segfault (e.g. adding long filenames using fileman.js)

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/d55686a5ef0d1f9706d17b18
Modified Files:
src/uifc/uifc32.c
Log Message:
A more complete fix for the upop() issue with long strings

Completely remove the 80 column screen width assumption/limit.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/ad84aa032681df2b5a7eb06d
Modified Files:
src/uifc/uifc32.c
Log Message:
Resolve MSVC warning

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/f55039de28c9dfc30450fec8
Modified Files:
src/uifc/uifc32.c
Log Message:
Only base drag-select highlight on the legacy attribute.

The fg/bg values all have multiple representations, and some bits
have extra meanings, so inequality there doesn't mean they're not
the same colour.

This fixes an issue where the *second* drag select on a screen
would not highlight blue backgrounds correctly. The background
colour gets the "pixels set" bit flown because of how
restorescreen() works, which caused this to "know" it's not blue
and use a blue background.

A (good) argument could be made that savescreen()/restorescreen()
shouldn't result in any changes, but since uifc32 doesn't directly
use colours that aren't represented by legacy attributes, there's
no need for it to get tricky here.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/c435be7c3f57a30fa0095e8e
Modified Files:
src/uifc/uifc32.c
Log Message:
Resolve GCC warning about not checking return value of vasprintf

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/8d530846c761b54828c61493
Modified Files:
src/uifc/uifc32.c
Log Message:
Change ugetstr() to be able to use modes K_FIND and K_CHANGES together

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/33e2888f435a8d48036759ae
Modified Files:
src/uifc/uifc32.c
Log Message:
Check for re-fill blk_scrn buffer before every use

Previously, the buffer was fixed to the default colours for the API
structure. Now of the fill char or colours change, the fill is
updated to reflect that.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/bbc3bb5a58375d21720b0eba
Modified Files:
src/uifc/uifc32.c
Log Message:
Address new warning from MSVC about comparison of signed-mismatched vars

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/bdab1d4d041d467c1d43c6d2
Modified Files:
src/uifc/uifc32.c
Log Message:
Don't modify the option list.

uifc would truncate space and control characters from the end of
options if the width was being auto-calculated.

This is a TERRIBLE idea because those are often const data.
If this is actually needed, it could be a weird strlen thing, but
it really shouldn't be needed.

This has been broken since the beginning.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/264210ef87b3800b5037fc62
Modified Files:
src/uifc/uifc32.c
Log Message:
uifc: showbuf width minimum is title_len + 8, not + 6

When the caller passes width < title_len + 6, showbuf clamped up to
title_len + 6. But the actual layout takes title_len + 8 cells -
two corners, two titlebreak chars, two surrounding spaces, plus a
minimum of two horizontal segments per side. The old minimum left titlebreak_right and top_right writing one cell each past the row's
end, corrupting the first two cells of row 2. Visible when the title
was longer than the longest line in the body buffer.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net