linux-kernel - Re: [PATCH] :scripts: prune-kernel : prunning kernel generalize way

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <c075bcce-0d6d-abee-7fb5-80821f2ae3a2@infradead.org>
Date:   Tue, 15 Oct 2019 16:03:04 -0700
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Bhaskar Chowdhury <unixbhaskar@...il.com>,
        yamada.masahiro@...ionext.com, michal.lkml@...kovi.net
Cc:     linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] :scripts: prune-kernel : prunning kernel generalize way

Subject: s/prunning/pruning/
Subject:  s/:scripts:/scripts:/

On 10/15/19 7:55 AM, Bhaskar Chowdhury wrote:
> This patch will remove old kernels from system selective way.

                                                 in a selective way.

> 
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@...il.com>
> ---
>  scripts/prune-kernel | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/scripts/prune-kernel b/scripts/prune-kernel
> index 74143f229f84..28fae6c91218 100755
> --- a/scripts/prune-kernel
> +++ b/scripts/prune-kernel
> @@ -33,7 +33,7 @@ printf "\t\n Enlist the installed kernels \n\n\n"
>  cd $boot_dir && pwd
> 
>  #Bash experts not recomend to run ls inside script,but you can by running that too
> -# ls -1 vmlinuz-*
> +# ls -1 vmlinuz-*
> 
>  #This is the recommended way , little complex but that's what people want to see!
> 
> @@ -42,10 +42,10 @@ find $boot_dir -name "vmlinuz-*" -type f -print0 -exec ls -1 {} \;
>  printf "\n\n\n Well, we need to purge some kernel to gain some space.\n\n\n"
> 
> 
> -printf "Please give the kernel version to remove: %s"
> +printf "Please give the kernel version to remove: %s"
>  read kernel_version
> 
> -remove_old_kernel
> +remove_old_kernel
> 
>  printf "\n\n Remove associated modules too ... \n\n"
> 
> @@ -54,7 +54,7 @@ cd $modules_dir && pwd
>  printf "\n\n\n Enlist the installed modules \n\n\n"
> 
>  #This is (-1) minus one not l(el)
> -# ls -1
> +# ls -1
> 
>  find $modules_dir -name "$kernel_version-*" -type f -print0 -exec ls -1 {} \;
> 
> @@ -66,14 +66,14 @@ rm -rf $modules_version
>  printf "\n\n Done \n\n"
> 
>  printf "\n\n Want to remove another?[Yn] : %s"
> -read response
> +read response
> 
> 
>  if [[ $response == "Y" ]]; then
> - printf "Please give another version to remove : %s"
> + printf "Please give another version to remove : %s"
>   read kernel_version
> 
> -remove_old_kernel
> +remove_old_kernel
> 
>  elif [[ $response == "n" ]]; then
> 
> @@ -81,4 +81,4 @@ elif [[ $response == "n" ]]; then
> 
>  fi
> 
> -exit 0
> +exit 0
> --
> 2.21.0
> 

Trying to apply this patch with 'patch --verbose' says:

checking file scripts/prune-kernel
Using Plan A...
Hunk #1 FAILED at 33.
Hunk #2 FAILED at 42.
Hunk #3 FAILED at 54.
Hunk #4 FAILED at 66.
Hunk #5 FAILED at 81.
5 out of 5 hunks FAILED
Hmm...  Ignoring the trailing garbage.
done


Those + and - lines with the same content don't make any sense.

And probably Cc: the author of the script:
J. Bruce Fields <bfields@...ldses.org>

-- 
~Randy