On Fri, 2 Aug 2019 22:40:01 +0800 Kang-Che Sung <[email protected]> wrote:
> What I suggested is to remove these lines:
>
> if (st.st_dev == rootdev) {
> // Show usage, it says new root must be a mountpoint
> bb_show_usage();
> }
>
> Because mount(..., MS_MOVE) will do the same check before moving the mount
> point, and the check would be done before deleting anything if your patch
> is applied. (And there is no point to show the program usage when it's
> already PID 1. You would rather see error message of what specifically goes
> wrong. The kernel will panic when PID 1 dies, leaving you no option but
> reboot.)
Thank you for your suggestion.
patch
Description: Binary data
_______________________________________________ busybox mailing list [email protected] http://lists.busybox.net/mailman/listinfo/busybox
