[ABRT PATCH] fix debuginfo installer expecting user input from a pipe - closes #696

Jakub Filak jfilak at redhat.com
Wed Sep 11 11:17:20 UTC 2013


According to commit b0d6d3dd99417a46e6ba641fef99235e78ead4d1 we should
modify abrt-action-analyze-core to create build_ids file with
umask(0002) or at least with read permission for others because it is
necessary to successfully
run /usr/libexec/abrt-action-install-debuginfo-to-abrt-cache which has
EUID="abrt's uid" and EGID="abrt's gid"


On Tue, 2013-09-10 at 14:57 +0200, Petr Kubat wrote:
> The debuginfo installer currently expects both build-ids and user input from a pipe with analaze-core, when run from an UI.
> We need to change it so that it passes the build-ids through a temporary file and only uses stdin to communicate with the user.
> 
> Signed-off-by: Petr Kubat <pkubat at redhat.com>
> ---
>  src/plugins/abrt-action-analyze-ccpp-local | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/plugins/abrt-action-analyze-ccpp-local b/src/plugins/abrt-action-analyze-ccpp-local
> index 0b02684..ef0976e 100755
> --- a/src/plugins/abrt-action-analyze-ccpp-local
> +++ b/src/plugins/abrt-action-analyze-ccpp-local
> @@ -24,10 +24,11 @@ if $INSTALL_DI; then
>      # debuginfo install fail even for root.
>      # Therefore, if we are root, we don't use the wrapper.
>      if [ x"`id -u`" = x"0" ]; then
> -        abrt-action-analyze-core --core=coredump | abrt-action-install-debuginfo --ids=- --size_mb=4096
> +        abrt-action-analyze-core --core=coredump -o build_ids && abrt-action-install-debuginfo --size_mb=4096
>      else
> -        abrt-action-analyze-core --core=coredump | /usr/libexec/abrt-action-install-debuginfo-to-abrt-cache --ids=- --size_mb=4096
> +        abrt-action-analyze-core --core=coredump -o build_ids && /usr/libexec/abrt-action-install-debuginfo-to-abrt-cache --size_mb=4096
>      fi
> +    rm -f build_ids
>  fi
>  
>  if [ $? = 0 ]; then




More information about the Crash-catcher mailing list