From 00754b35a22d491b614fefe5149bcedad1523ef2 Mon Sep 17 00:00:00 2001 From: Arusekk Date: Thu, 9 Oct 2025 08:48:35 +0200 Subject: devdraw: fix dangling else Because of a dangling else, after adding a name to an image with 'N', it was immediately deleted, resulting in Enoname in response to 'n'. This went mostly unnoticed, since plan9port itself does not use named images, but it can break external applications like truedraw[1]. [1]: https://git.sr.ht/~arusekk/truedraw Fixes: c66b52501b63 ("new draw server") --- src/cmd/devdraw/devdraw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cmd/devdraw/devdraw.c b/src/cmd/devdraw/devdraw.c index 234cdf1b..fe02dc8c 100644 --- a/src/cmd/devdraw/devdraw.c +++ b/src/cmd/devdraw/devdraw.c @@ -1107,10 +1107,10 @@ draw_datawrite(Client *client, void *v, int n) goto Enodrawimage; if(di->name) goto Enamed; - if(c) + if(c){ if(drawaddname(client, di, j, (char*)a+7, &err) < 0) goto error; - else{ + }else{ dn = drawlookupname(client, j, (char*)a+7); if(dn == nil) goto Enoname; -- cgit v1.2.3