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: <20181203051806.GA6639@thunk.org>
Date:   Mon, 3 Dec 2018 00:18:06 -0500
From:   "Theodore Y. Ts'o" <tytso@....edu>
To:     Gabriel Krisman Bertazi <krisman@...labora.com>
Cc:     kernel@...labora.com, linux-ext4@...r.kernel.org
Subject: Re: [PATCH e2fsprogs v4 0/9] Support encoding awareness and casefold

Hi Gabriel,

Thanks for all of your work on it so far.  There's still some work
that we need to do, but I think this is good enough for merge what we
have, so I've pulled this into the e2fsprogs master and next branches.
There were a few commit description adjustments, and in one place I
replaced a call to calloc() with a fixed-size buffer to just using a
on-stack buffer.

The main thing I noticed is that nothing is initializing fs->encoding.
We need to do that in ext2fs_open() and ext2fs_initialize().  We also
need to add some test cases --- if we did, we would have noticed the
missing initialization of fs->encoding.

I do want to include of the script which generates utf8data.h in
e2fsprogs sources; we'll need to it in sync with the kernel sources,
and so long as it stays in sync, we can always double check that
version of utf8data.h in e2fsprogs matches the version the one in the
kernel.  But for reasons of GPL compliance, we should keep a copy of
the script which creates the generated .h file.

Again, thanks for the work so far!

					- Ted

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ