Fix uninitialized t_l_req in --verbose FD mode

Move gettimeofday(&t_l_req) from command-mode-only block to just
before the flock() loop, so it covers both command and FD paths.
Previously, FD mode with --verbose read uninitialized stack memory,
producing garbage like "took 1774858164251193 microseconds".

Add regression test verifying verbose FD mode timing is sane (<1s).

Author: voltechs

src/flock.c

@@ -265,10 +265,8 @@ int main(int argc, char *argv[]) {
 			open_flags = O_WRONLY | O_NOCTTY | O_CREAT;
 		}
 
-		if (verbose) {
-			gettimeofday(&t_l_req, NULL);
+		if (verbose)
 			printf("flock: getting lock\n");
-		}
 		fd = open(filename, open_flags, 0666);
 
 		// directories don't like O_WRONLY (and sometimes O_CREAT)
@@ -314,6 +312,9 @@ int main(int argc, char *argv[]) {
 			err(EX_OSERR, "could not set interval timer");
 	}
 
+	if (verbose)
+		gettimeofday(&t_l_req, NULL);
+
 	while (0 != flock(fd, type | block)) {
 		switch (errno) {
 		case EWOULDBLOCK: // non-blocking lock not available

t/default.bats

@@ -618,6 +618,16 @@ get_elapsed() {
 	[[ "$result" == *"microseconds"* ]]
 }
 
+@test "--verbose in fd mode reports sane timing" {
+	# Regression: t_l_req was uninitialized in FD mode, producing
+	# garbage like "took 1774858164251193 microseconds"
+	result=$( (${FLOCK} --verbose 8) 8> ${LOCKFILE} 2>&1 )
+	[[ "$result" == *"microseconds"* ]]
+	# Extract the number and verify it's under 1 second (1000000 us)
+	elapsed=$(echo "$result" | grep -o '[0-9]* microseconds' | grep -o '[0-9]*')
+	[ "$elapsed" -lt 1000000 ]
+}
+
 ###############################################################################
 # Lock is released when holder exits
 ###############################################################################