[devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Denis Smirnov]

Так что мне делать-то? Или encfs никому кроме меня не нужен?

Понять в чем проблема и как ее исправлять мне слабо.


---------- Forwarded message ----------
From: ALT beekeeper <beehive@altlinux.org>
Date: 2010/3/10
Subject: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]
To: mithraen@altlinux.org
Копия: beehive@altlinux.org

Testing interfaces
AES, key length 128, block size 256:  terminate called after throwing
an instance of 'boost::archive::archive_exception'
 what():  stream error
/usr/src/tmp/rpm-tmp.51960: line 127: 19515 Aborted                 encfs/test
error: Bad exit status from /usr/src/tmp/rpm-tmp.51960 (%build)

RPM build errors:
   Bad exit status from /usr/src/tmp/rpm-tmp.51960 (%build)
Command exited with non-zero status 1
49.65user 10.77system 1:13.42elapsed 82%CPU (0avgtext+0avgdata 0maxresident)k
0inputs+0outputs (0major+2100033minor)pagefaults 0swaps
hsh-rebuild: rebuild of `fuse-encfs-1.5.2-alt7.src.rpm' failed.
Command exited with non-zero status 1

--
Sincerely yours,
ALT beekeeper

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 498 bytes --]

On Thu, Mar 11, 2010 at 03:49:31PM +0300, Denis Smirnov wrote:
> Так что мне делать-то? Или encfs никому кроме меня не нужен?
> 
> Понять в чем проблема и как ее исправлять мне слабо.

Апстриму отрепортить не пробовал?

> Testing interfaces
> AES, key length 128, block size 256:  terminate called after throwing
> an instance of 'boost::archive::archive_exception'
>  what():  stream error
> /usr/src/tmp/rpm-tmp.51960: line 127: 19515 Aborted                 encfs/test


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 485 bytes --]

On Thu, Mar 11, 2010 at 04:03:54PM +0300, Dmitry V. Levin wrote:

DVL> Апстриму отрепортить не пробовал?

Проблема вылезла после обновления boost'а у нас.

Притом я, честно говоря, совсем не в состоянии понять что там происходит,
ибо C++ знаю на уровне "С с классами" и дать сколь-нибудь внятный репорт.

Может просто линковаться со старым boost?

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Vladislav Zavjalov]

On Thu, Mar 11, 2010 at 08:52:05PM +0300, Денис Смирнов wrote:
> On Thu, Mar 11, 2010 at 04:03:54PM +0300, Dmitry V. Levin wrote:
> 
> DVL> Апстриму отрепортить не пробовал?
> 
> Проблема вылезла после обновления boost'а у нас.
> 
> Притом я, честно говоря, совсем не в состоянии понять что там происходит,
> ибо C++ знаю на уровне "С с классами" и дать сколь-нибудь внятный репорт.
> 
> Может просто линковаться со старым boost?

Покопался...

boost::archive используется для чтения/записи xml в FileUtils.cpp

Облом происходит в std::istream &operator >>
в test.cpp:230. Записать xml оно сумело, а прочитать - нет.

А прочитать оно его не сумело, поскольку была выставлена версия класса
20080816 (см BOOST_CLASS_VERSION в FileUtils.cpp), а в xml попало 26800 (а для
старых версий она хочет читать что-то другое).

Взяв минимальный пример с
http://www.boost.org/doc/libs/1_42_0/libs/serialization/doc/index.html
устроил такую штуку, показывающую, что с большими версиями классов
происходит фигня, а именно, они устанавливаются по модулю 65536:

Так что проблема в boost, и как ее хорошо обойти  в fuse-encfs -
непонятно...

===

#include <iostream>

#include <boost/archive/text_oarchive.hpp>
#include <boost/archive/text_iarchive.hpp>

#define VERSION 65536

struct test{
    template<class Archive>
    void serialize(Archive & ar, const unsigned int version)
    {
        std::cerr << "\n" << "new_version: " << version << "\n"
                       << "old_version: " << VERSION << "\n";
        ar & i;
    }
    int i;
};

BOOST_CLASS_VERSION(test, VERSION)

int main() {
    test var;
    var.i=100;

    boost::archive::text_oarchive oa(std::cout);
    oa << var;
}

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Vladislav Zavjalov]

[-- Attachment #1: Type: text/plain, Size: 144 bytes --]

> Так что проблема в boost, и как ее хорошо обойти  в fuse-encfs -
> непонятно...

Придумал не слишком красивенький обход (в приложении)

Слава

[-- Attachment #2: 0001-boost-serialization-version-workaround.patch --]
[-- Type: text/plain, Size: 2267 bytes --]

>From 37e22a7215e7cd68aa5ee49a996bbed302b7c25e Mon Sep 17 00:00:00 2001
From: Vladislav Zavjalov <slazav@altlinux.org>
Date: Fri, 12 Mar 2010 03:44:21 +0300
Subject: [PATCH] boost::serialization::version workaround

---
 fuse-encfs/encfs/FileUtils.cpp |   18 +++++++++++++++++-
 1 files changed, 17 insertions(+), 1 deletions(-)

diff --git a/fuse-encfs/encfs/FileUtils.cpp b/fuse-encfs/encfs/FileUtils.cpp
index 73a589f..9e69b57 100644
--- a/fuse-encfs/encfs/FileUtils.cpp
+++ b/fuse-encfs/encfs/FileUtils.cpp
@@ -131,6 +131,10 @@ namespace boost
         void save(Archive &ar, const EncFSConfig &cfg, 
                 unsigned int version)
         {
+	    //workaround: some boost versions wraps version to 65536!
+            //works for years 2006..2012
+	    if (version < 65536) version+=65536*306;
+
             (void)version;
             ar << make_nvp("creator", cfg.creator);
             ar << make_nvp("cipherAlg", cfg.cipherIface);
@@ -161,6 +165,10 @@ namespace boost
         template<class Archive>
         void load(Archive &ar, EncFSConfig &cfg, unsigned int version)
         {
+	    //workaround: some boost versions wraps version to 65536!
+            //works for years 2006..2012
+	    if (version < 65536) version+=65536*306;
+
             cfg.subVersion = version;
             ar >> make_nvp("creator", cfg.creator);
             ar >> make_nvp("cipherAlg", cfg.cipherIface);
@@ -207,12 +215,20 @@ namespace boost
         template<class Archive>
         void serialize(Archive &ar, EncFSConfig &cfg, unsigned int version)
         {
+	    //workaround: some boost versions wraps version to 65536!
+            //works for years 2006..2012
+	    if (version < 65536) version+=65536*306;
+
             split_free(ar, cfg, version);
         }
 
         template<class Archive>
-        void serialize(Archive &ar, Interface &i, const unsigned int version)
+        void serialize(Archive &ar, Interface &i, unsigned int version)
         {
+	    //workaround: some boost versions wraps version to 65536!
+            //works for years 2006..2012
+	    if (version < 65536) version+=65536*306;
+
             (void)version;
             ar & make_nvp("name", i.name());
             ar & make_nvp("major", i.current());
-- 
1.6.5.3

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 395 bytes --]

On Fri, Mar 12, 2010 at 12:58:06AM +0300, Vladislav Zavjalov wrote:

VZ> Придумал не слишком красивенький обход (в приложении)

Спасибо! А что вообще этот кусок кода в encfs делает, и есть ли риск
несовместимостей от этого патча?

Я могу его слепо приложить?

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Vladislav Zavjalov]

On Fri, Mar 12, 2010 at 06:21:06PM +0300, Денис Смирнов wrote:
> On Fri, Mar 12, 2010 at 12:58:06AM +0300, Vladislav Zavjalov wrote:
> 
> VZ> Придумал не слишком красивенький обход (в приложении)
> 
> Спасибо! А что вообще этот кусок кода в encfs делает, и есть ли риск
> несовместимостей от этого патча?

Я это понял так:
Это запись и чтение xml-конфигов с помощью boost/archive и boost/serialization
В xml записывается объект типа EncFSConfig, и к нему пишется
версия, чтоб потом правильно читать устаревшие конфиги.

Фигня в том, что версия устанавливается 20080816, а где-то в boost-е 
она приводится к uint16 (я покопался, но не понял где - везде вроде unsigned int).
А при чтении там стоят всякие проверки, типа "если старше 20080816 - такие поля не нужны" 
и т.п. И с учетом этих проверок boost'у предлагают читать не
тот объект, который записан. (Это все можно смотреть в файле FileUtils.cpp)

Насчет патча - он сомнительный. Он перед всякими проверками говорит
if (version < 65536) version+=65536*306;
Мало того, что с версиями > 20120101 это уже не будет работать, так еще
и конфиги, записанные разными весиями программы могут перепутаться.
Сейчас в конфиги будет писаться неправильная обрезанная версия
и читаться она будет правильно только сборкой с этим патчем.

В общем, я бы еще подумал. Как минимум, попытался бы
сделать, чтоб в конфиги писалась правильная версия (хотя это какие-то
недра boost'a)...

И - попинать знатоков boost'a, что там у них за фигня.

Слава

Re: [devel] Fwd: fuse-encfs-1.5.2-alt7: Sisyphus/x86_64 test rebuild failed [3]

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 722 bytes --]

On Fri, Mar 12, 2010 at 06:49:58PM +0300, Vladislav Zavjalov wrote:

VZ> Насчет патча - он сомнительный. Он перед всякими проверками говорит
VZ> if (version < 65536) version+=65536*306;
VZ> Мало того, что с версиями > 20120101 это уже не будет работать, так еще
VZ> и конфиги, записанные разными весиями программы могут перепутаться.
VZ> Сейчас в конфиги будет писаться неправильная обрезанная версия
VZ> и читаться она будет правильно только сборкой с этим патчем.

Получается что если этот патч сейчас приложить -- его придется иметь
пожизненно :(

Тогда подождем свежего boost.

-- 
С уважением, Денис

http://mithraen.ru/
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]