Conan-center-index: [fmtlib] <fmt>/<6.0.0>: test_package.cpp bug

Created on 1 Oct 2019 · 8Comments · Source: conan-io/conan-center-index

Originally reported to fmt repo here...

https://github.com/fmtlib/fmt/issues/1339

Package and Environment Details (include every applicable attribute)

Package Name/Version: fmt/6.0.0
Operating System+version: Windows
Compiler+version: vs-15
Conan version: conan 1.14.4
Python version: Python 3.7.0

It appears the test_package.cpp is incorrectly using the buffer.data() in it's print statement on line 45...

fmt::print("Euler number: {}\n", buf.data());

This is causing a non-null-terminated output such as...

Euler number: 2.71828╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠╠╠╠╠╠╠☺⌂Bç ┐♣@P∙O_∩

fmt-test-package-issue.zip

bug

Source

keysight-daryl

👍1

Most helpful comment

I guess issue is that fmt::memory_buffer::data is not guaranteed to be null-terminated, so it's UB to output it as a string. we have to compose std::string from fmt::memory_buffer::data and fmt::memory_buffer::size in order to have null-terminated string.

doc

The output can be converted to an std::string with to_string(out).

SSE4 on 2 Oct 2019

👍2

All 8 comments

@keysight-daryl !

It's working normal for me. It can be related to your terminal. Which terminal do you use? Also, are you really running Conan 1.14.4?

uilianries on 1 Oct 2019

doc

The output can be converted to an std::string with to_string(out).

SSE4 on 2 Oct 2019

👍2

yes, conan 1.14.4 is the version we are locked at here at Keysight. I think the fmt dev indicated that the data() function is not intended to return a properly terminated string is the issue. It only caused a problem for debug builds in VS2017 but I supposed that it doesn't cause a problem otherwise is just a case of luck. I believe it should be changed to call to_string() instead.

keysight-daryl on 2 Oct 2019

It might be related to the fact that just recently fmt is supporting non string data in it's buffers.

keysight-daryl on 2 Oct 2019

Reviewing the proposed fix I believe it still has a bug. According to the docs to convert to a string is to call to_string(buf) not to_string(buf.data())

keysight-daryl on 2 Oct 2019

❤1

I would also like to recommend updating of the conanfile to properly package/import the associated *.pdb files for debug builds so that consumers avoid the 'missing PDB' compiler warnings. For example...

`
def imports(self):
self.copy('*.pdb', dst='bin', src='bin')

def package(self):
    self.copy("LICENSE.rst", dst="licenses", src=self._source_subfolder)
    if self.options.header_only:
        src_dir = os.path.join(self._source_subfolder, "src")
        header_dir = os.path.join(self._source_subfolder, "include")
        dst_dir = os.path.join("include", "fmt")
        self.copy("*.h", dst="include", src=header_dir)
        self.copy("*.cc", dst=dst_dir, src=src_dir)
    else:
        cmake = self._configure_cmake()
        cmake.install()
        self.copy('*.pdb', dst='bin', keep_path=False)

keysight-daryl on 2 Oct 2019

PDB files aren't allowed for CCI recipes https://github.com/conan-io/conan-center-index/wiki/Error-Knowledge-Base#kb-h017-pdb-files-not-allowed