Re: [devel] [i586] mailutils & glibc 2.26/2.27

["Ivan A. Melnikov" <iv@altlinux.org>]

[-- Attachment #1.1: Type: text/plain, Size: 990 bytes --]

On Wed, Aug 15, 2018 at 02:13:46PM +0400, Sergey Afonin wrote:
> Продолжаю про слом теста. 
> 
> Про mailutils 3.4 выяснилось следующее. При сборке с
> glibc-devel-6:2.25-alt4 все тесты проходят, если же
> обновить до glibc-devel-6:2.26.0.124.98f244e-alt1, то
> получается 
> 
> 189: mhl usenet                 FAILED (mhl.at:49)
> 
> Пересборка на i586 сломалась, соответственно, 26-ого
> декабря. Кто виноват, и что делать?
> 

Не знаю, но предлагаю разобраться и починить.

В моём хешере приложенный патч вроде бы исправляет проблему, и он явно не
делает ничего плохого, но я не уверен, что решать проблему надо именно
так.

--
  wbr,
    iv m.

[-- Attachment #1.2: mailutils-3.4-alt-fix-mh-format.patch --]
[-- Type: text/x-patch, Size: 800 bytes --]

commit 51f10424d329aad87136ef8d7fac2cae3ce766df
Author: Ivan A. Melnikov <iv@altlinux.org>
Date:   Wed Aug 15 16:08:55 2018 +0400

    Force terminating \0 in mh_format result
    
    Apparently, mu_stream_read is not required to put the
    terminating zero into the buffer, so we need to do it
    in its caller.
    
    Signed-off-by: Ivan A. Melnikov <iv@altlinux.org>

diff --git a/mh/mh_format.c b/mh/mh_format.c
index d4aff7f4d..77faaf030 100644
--- a/mh/mh_format.c
+++ b/mh/mh_format.c
@@ -726,6 +726,7 @@ mh_format_str (mh_format_t fmt, char *str, size_t width, char **pstr)
   buf = mu_alloc (size + 1);
   MU_ASSERT (mu_stream_seek (outstr, 0, MU_SEEK_SET, NULL));
   MU_ASSERT (mu_stream_read (outstr, buf, size, NULL));
+  buf[size] = 0;
 
   *pstr = buf;
   

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]