Skip to contentSkip to conversation

New: Adding custom user account does nothing when the user table text file dose not have a terminating newline

Buildroot [email protected] 1 participant 4 messages Aug 30, 2023 – Aug 31, 2023

lore
Share & tools
lore
#0 lore raw

https://bugs.busybox.net/show_bug.cgi?id=15745

Bug ID: 15745
Summary: Adding custom user account does nothing when the user
table text file dose not have a terminating newline
Product: buildroot
Version: 2023.02.2
Hardware: PC
OS: Linux
Status: NEW
Severity: normal
Priority: P5
Component: Other
Assignee: [email protected]
Reporter: [email protected]
CC: [email protected]
Target Milestone: ---

As title

For example,

```text @ package/bar/users_table.txt
foo -2 foo -2 =foo /home/foo /bin/bash - Foo user
```

When configure buildroot to use this user table according to the manual, log shows that the content is being copied to the master user table:

_________________________________________________________________________

Quoted 3 lines

cat package/bar/users_table.txt >> /home/yanghao/open-source/buildroot/buildroot/output/build/buildroot-fs/full_users_table.txt

/yanghao/open-source/buildroot/buildroot/support/scripts/mkusers /home/yanghao/open-source/buildroot/buildroot/output/build/buildroot-fs/full_users_table.txt /home/yanghao/open-source/buildroot/buildroot/output/build/buildroot-fs/ext2/target >> /home/yanghao/open-source/buildroot/buildroot/output/build/buildroot-fs/ext2/fakeroot

__________________________________________________________________________

However, no user is created in the resulting filesystem.

Adding a newline at the end fixes it and works as expected.

```text
foo -2 foo -2 =foo /home/foo /bin/bash - Foo user

```

In the end, I think this is a bug because it is not mentioned in the manual Chapter 26. Makeusers syntax documentation and 9.6. Adding custom user accounts.

Signature · 6 lines
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#1 lore raw

[Buildroot] [Bug 15745] Adding custom user account does nothing when the user table text file dose not have a terminating newline

https://bugs.busybox.net/show_bug.cgi?id=15745

--- Comment #1 from Edgar Bonet <[email protected]> ---

Quoted 1 line

Adding a newline at the end fixes it and works as expected.

Do you mean that the file should end with *two* newline characters?

Note that, in a Unix environment, a text file is by definition a sequence of LF-terminated lines of text. This implies that a non-empty file that does not end with an LF character is not a valid text file.

The manual does mention that users tables are “regular text files”. If the user account creation fails to parse an invalid text file, I would not consider it a bug.

Signature · 6 lines
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#2 lore raw

[Buildroot] [Bug 15745] Adding custom user account does nothing when the user table text file dose not have a terminating newline

In reply to [email protected]

https://bugs.busybox.net/show_bug.cgi?id=15745

--- Comment #2 from [email protected] --- Okay... I meant only one LF is needed.

I've to admit that I don't know the definition exactly although I have seen software that marks non-LF terminated files as questionable.

Thank you for letting me know and I will close this ticket.

Signature · 6 lines
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#3 lore raw

[Buildroot] [Bug 15745] Adding custom user account does nothing when the user table text file dose not have a terminating newline

In reply to [email protected]

https://bugs.busybox.net/show_bug.cgi?id=15745

[email protected] changed:

What |Removed |Added
----------------------------------------------------------------------------

         Resolution|---                         |INVALID
             Status|NEW                         |RESOLVED
Signature · 6 lines
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot