From 3121eeb5e68bedd6e2398f35b9bd60b69709a47e Mon Sep 17 00:00:00 2001
From: Josselin Poiret <dev@jpoiret.xyz>
Date: Tue, 14 Dec 2021 18:49:54 +0100
Subject: [PATCH] Replace hardcoded command length

---
 src/screenshot/screenshot.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/screenshot/screenshot.c b/src/screenshot/screenshot.c
index 9df0ea7..5f27681 100644
--- a/src/screenshot/screenshot.c
+++ b/src/screenshot/screenshot.c
@@ -42,8 +42,9 @@ static bool exec_screenshooter_interactive(const char *path) {
 		perror("fork");
 		return false;
 	} else if (pid == 0) {
-		char cmd[strlen(path) + 25];
-		snprintf(cmd, sizeof(cmd), "grim -g \"$(slurp)\" -- %s", path);
+		const char cmd_fmt[] = "grim -g \"$(slurp)\" -- %s";
+		char cmd[strlen(path) + sizeof(cmd_fmt)];
+		snprintf(cmd, sizeof(cmd), cmd_fmt, path);
 		execl("/bin/sh", "/bin/sh", "-c", cmd, NULL);
 		perror("execl");
 		exit(127);
-- 
2.34.0

right'></td></tr></table>
<table class='tabs'><tr><td>
<a href='/guix/about/'>about</a><a href='/guix/'>summary</a><a href='/guix/refs/?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>refs</a><a class='active' href='/guix/log/tests/gnu-maintenance.scm'>log</a><a href='/guix/tree/tests/gnu-maintenance.scm?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>tree</a><a href='/guix/commit/tests/gnu-maintenance.scm?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>commit</a><a href='/guix/diff/tests/gnu-maintenance.scm?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>diff</a></td><td class='form'><form class='right' method='get' action='/guix/log/tests/gnu-maintenance.scm'>
<input type='hidden' name='id' value='6c34d33cde04a75ce4874ea037afcb20af4673be'/><select name='qt'>
<option value='grep'>log msg</option>
<option value='author'>author</option>
<option value='committer'>committer</option>
<option value='range'>range</option>
</select>
<input class='txt' type='search' size='10' name='q' value=''/>
<input type='submit' value='search'/>
</form>
</td></tr></table>
<div class='path'>path: <a href='/guix/log/?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>root</a>/<a href='/guix/log/tests?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>tests</a>/<a href='/guix/log/tests/gnu-maintenance.scm?id=6c34d33cde04a75ce4874ea037afcb20af4673be'>gnu-maintenance.scm</a></div><div class='content'><table class='list nowrap'><tr class='nohover'><th class='left'>Age</th><th class='left'>Commit message (<a href='/guix/log/tests/gnu-maintenance.scm?id=6c34d33cde04a75ce4874ea037afcb20af4673be&amp;showmsg=1'>Expand</a>)</th><th class='left'>Author</th></tr>
<tr><td><span title='2023-09-27 23:38:21 +0200'>2023-09-27</span></td><td><a href='/guix/commit/tests/gnu-maintenance.scm?id=72fc91b113564f8006094c0118f06af0f414d21b'>tests: ‘rewrite-url’ test no longer depends on network access.</a><span class='msg-avail'>...<span class='msg-tooltip'>Fixes &lt;https://issues.guix.gnu.org/66097&gt;.
Previously, this test would actually need to access dist.libuv.org.

* tests/gnu-maintenance.scm ("rewrite-url, without to-version"): Change
URLs from https to http, and from libuv.org to libuv.example.org.
Parameterize ‘current-http-proxy’.

Reported-by: Janneke Nieuwenhuizen &lt;janneke@gnu.org&gt;
</span></span></td><td>Ludovic Courtès</td></tr>