chore(deps): bump actions/upload-artifact in /.github/workflows

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

.github/workflows/build.yml

@@ -17,7 +17,7 @@ jobs:
       - id: build
         run: ARCH=amd64 assets/github_scripts/build.sh
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-amd64
           path: lidm-amd64
@@ -42,7 +42,7 @@ jobs:
       - id: build
         run: ARCH=i386 assets/github_scripts/build.sh CFLAGS="-O3 -Wall -m32"
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-i386
           path: lidm-i386
@@ -70,7 +70,7 @@ jobs:
       - if: always()
         run: set +e; cat gss.out >>"$GITHUB_STEP_SUMMARY"; cat go.out >>"$GITHUB_OUTPUT"
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-aarch64
           path: lidm-aarch64
@@ -98,7 +98,7 @@ jobs:
       - if: always()
         run: set +e; cat gss.out >>"$GITHUB_STEP_SUMMARY"; cat go.out >>"$GITHUB_OUTPUT"
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-armv7
           path: lidm-armv7
@@ -126,7 +126,7 @@ jobs:
       - if: always()
         run: set +e; cat gss.out >>"$GITHUB_STEP_SUMMARY"; cat go.out >>"$GITHUB_OUTPUT"
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-riscv64
           path: lidm-riscv64
@@ -155,7 +155,7 @@ jobs:
       - if: always()
         run: set +e; cat gss.out >>"$GITHUB_STEP_SUMMARY"; cat go.out >>"$GITHUB_OUTPUT"
 
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v7
         with:
           name: build-amd64-musl
           path: lidm-amd64-musl

CHANGELOG.md

@@ -4,6 +4,10 @@
 <!-- By "very relevant" I mean big features or something manual packagers should know, like leftover files -->
 <!-- Once a release would be opened, group the last bunch of dangling changes, add release version as header and its date -->
 
+# 2.0.2
+
+- fix off-by-one error with launch state save
+
 # 2.0.1
 
 - source and header files can be nested in `src/` and `include/`

Makefile

@@ -1,4 +1,4 @@
-VERSION := 2.0.1
+VERSION := 2.0.2
 .DEFAULT_GOAL := lidm
 
 CDIR = src

README.md

@@ -34,7 +34,7 @@ But [according to farouk](https://github.com/javalsai/lidm/issues/91#issuecommen
 - Simple as C, meant to depend only on standard libc and basic unix system headers.
 - Fully customizable: ALL strings, colors (with its ANSI keys) and most behavior.
 - Experimental Xorg support[\*](https://github.com/javalsai/lidm/pull/80?#issuecomment-3764073217) and wayland sessions, while supporting the default user shell (if enabled in config)
-- Init agnostinc (systemd, dinit, runit, openrc and s6).
+- Init agnostic (systemd, dinit, runit, openrc and s6).
 - Supports [fido yubikeys](./docs/yubikey.md) (via pam_u2f).
 
 # Table of Contents

assets/pkg/nix/module.nix

@@ -11,7 +11,7 @@ let
   dmcfg = config.services.displayManager;
   desktops = dmcfg.sessionData.desktops;
 
-  version = "2.0.0";
+  version = "2.0.1";
   lidmPkg = pkgs.callPackage ./lidm.nix {
     inherit pkgs;
     config = {
@@ -21,7 +21,7 @@ let
         owner = "javalsai";
         repo = "lidm";
         rev = "v${version}";
-        sha256 = "sha256-dI1OGndbT6wFAhuGmsPZPqLFvtPREfO/3HqhmlSMpN4=";
+        sha256 = "sha256-bpUqhD1JSiYRf7w7ylEMXHMvEpnSri1zZSxRQPdZWB4=";
       };
 
       xsessions = "${desktops}/share/xsessions";

src/launch_state.c

@@ -22,16 +22,16 @@ int read_launch_state(struct LaunchState* NNULLABLE state) {
 
   size_t num = 0;
   ssize_t chars = getline(&state->username, &num, state_fd);
-  if (chars < 0) goto fail;
-  if (state->username[chars] == '\n') state->username[chars] = 0;
+  if (chars <= 0) goto fail;
+  if (state->username[chars - 1] == '\n') state->username[chars - 1] = 0;
 
   num = 0;
   chars = getline(&state->session_opt, &num, state_fd);
-  if (chars < 0) {
+  if (chars <= 0) {
     free(state->session_opt);
     goto fail;
   }
-  if (state->session_opt[chars] == '\n') state->session_opt[chars] = 0;
+  if (state->session_opt[chars - 1] == '\n') state->session_opt[chars - 1] = 0;
 
   (void)fclose(state_fd);
   return 0;