Commit Briefs

David du Colombier

troff: import HB, HI and HX fonts from Plan 9

These font were accidentally left out, because their name conflicted with Hb, Hi and Hx fonts on case-insensitive file systems. These fonts were later renamed to longer names. Fixes #84. Change-Id: Ia0631f270a8cee6bee4337c4557837bba1a405ab Reviewed-on: https://plan9port-review.googlesource.com/2900 Reviewed-by: Russ Cox <rsc@swtch.com>


Gleydson Soares

factotum: Fix an operator precedence bug in the secstore code

`==` is higher precedence than `&`. Parenthesize. Change-Id: I2c7ee588fea0d8a66e1c8424f26630015388d61a Signed-off-by: Dan Cross <cross@gajendra.net> Reviewed-on: https://plan9port-review.googlesource.com/2860 Reviewed-by: Gleydson Soares <gsoares@gmail.com>


Russ Cox

yacc: Fix a bug from at least 1995.

Fix a bug folding newlines in strings constants in C code snippets in YACC. This code has existed since at least 2nd Edition Plan 9. Change-Id: Iba17b89a6529ac9fa6610bf0b44f551904174c26 Signed-off-by: Dan Cross <cross@gajendra.net> Reviewed-on: https://plan9port-review.googlesource.com/2840 Reviewed-by: Russ Cox <rsc@swtch.com>


Gleydson Soares

libdraw: replace hand-rolled realloc, preventing buffer overflow.

The original buffer is f->nsubf*sizeof *subf bytes (oldsize) large. Once it's full, a new buffer of (f->nsubf+DSUBF)*sizeof *subf (newsize) is mallocated. Unfortunately memmove() reads (newsize) bytes from the original (oldsize) buffer, causing a buffer overflow. By switching to realloc(), we don't need to do buffer size calculation, memmoving, and freeing of the original buffer. Change-Id: Ibf85bc06abe1c8275b11acb1d7d346a14291d2cd Reviewed-on: https://plan9port-review.googlesource.com/1520 Reviewed-by: Gleydson Soares <gsoares@gmail.com>


Gleydson Soares

9term: Add missing parentheses, preventing buffer overflow.

(el-sr) is the string length and (sizeof wdir - strlen(name) - 20) is the buffer size. When the string length is greater than the buffer size, the beginning of the string is supposed to be trimmed to fit in the buffer size. Unfortunately a pair of parentheses were missing, pointing sr outside the buffer, and the for loop below then reads outside the buffer. For certain binary data printed in a window, it causes a segfault. Change-Id: Iffeaa348260ee2a5a36d9577308fb8d1c1688d05 Reviewed-on: https://plan9port-review.googlesource.com/1540 Reviewed-by: Gleydson Soares <gsoares@gmail.com>


Russ Cox

9pfuse: support osxfuse>=3.3.0

Change-Id: Ibca68261016b0f588e169e9f319748b5ba7d15ac Reviewed-on: https://plan9port-review.googlesource.com/2700 Reviewed-by: michaelian ennis <michaelian.ennis@gmail.com> Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

9term: c nits

Change-Id: Ia18ebb28f24bc69f404004896f9ce54ddb53faf9 Reviewed-on: https://plan9port-review.googlesource.com/2782 Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

cmd/9term: A hack because I'm constantly getting confused about ^C vs fn+delete

Since Google (and a lot of the outside) is so engrained with using ^C as interrupt, I'd like to be able to use it in 9term if I've stty'd my intr to ^C. Without this, hitting ^C still works but if the program behind the window isn't reading from /dev/cons, it won't take effect till after I hit a newline which is often very confusing. I know this is a hack since it only works if I stty intr ^C but that seems the only other character that gets used anyways. Change-Id: I0597e63b2d7628f5668c648e6dba6f281e4b27fd Reviewed-on: https://plan9port-review.googlesource.com/2742 Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

src/cmd: take fontsrv out of BUGGERED

Change-Id: Iac5726f13039fba818536dc375d38cc3e27aca3a Reviewed-on: https://plan9port-review.googlesource.com/2741 Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

all: fix or silence all INSTALL warnings on macOS

Should be a clean build now. Change-Id: Id3460371cb5e8d4071f8faa9c2aec870d213a067 Reviewed-on: https://plan9port-review.googlesource.com/2781 Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

9term, win: work around bsd linker nonsense

Change-Id: Ifcef0636ee1e1fd0f9b06a8d1a99d58fae831318 Reviewed-on: https://plan9port-review.googlesource.com/2780 Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

9term, mc: conspire to handle hidpi displays

9term now uses the low bit of ws.ws_ypixel to signal whether this is a hidpi display, and mc adjusts the font it uses for columnation accordingly. Makes 'lc' work right on hidpi displays. Change-Id: I52928871ffb7f4c6fd6722f3d59f1836379148c6 Reviewed-on: https://plan9port-review.googlesource.com/2760 Reviewed-by: Russ Cox <rsc@swtch.com>


David du Colombier

include: define _DEFAULT_SOURCE

Since glibc 2.20, the _BSD_SOURCE and _SVID_SOURCE macros are deprecated in favor of the _DEFAULT_SOURCE macro. See https://sourceware.org/glibc/wiki/Release/2.20#Packaging_Changes Change-Id: I18dd6a698f3f5aa51d1e45bf53b031bb061e17e8 Reviewed-on: https://plan9port-review.googlesource.com/1500 Reviewed-by: David du Colombier <0intro@gmail.com>


Russ Cox

libregexp: various fixes

- Check before dereference. - Fix memory leak - Remove duplicate if statement. - Remove trailing spaces. Change-Id: I140a05c8f7006493dddae753aebfa5b4577c65ef Reviewed-on: https://plan9port-review.googlesource.com/1301 Reviewed-by: Neven Sajko <nsajko@gmail.com> Reviewed-by: Russ Cox <rsc@swtch.com>


Russ Cox

fix segfaults on OpenBSD: int cannot hold all the values a ptrdiff_t can take.

Change-Id: I90806e6f2e46a94f38aa0a8a7038aa6599b448b3 Reviewed-on: https://plan9port-review.googlesource.com/2660 Reviewed-by: Russ Cox <rsc@swtch.com>