Uploaded image for project: 'ICU'
  1. ICU
  2. ICU-20001

number_decimfmtprops.cpp has unaligned static memory

    Details

      Description

      The file has the following code:

        char kRawDefaultProperties[sizeof(DecimalFormatProperties)];
  ...
  void U_CALLCONV initDefaultProperties(UErrorCode&) {
      new(kRawDefaultProperties) DecimalFormatProperties();
  }

      We have a report that this causes a crash on ARM. The DecimalFormatProperties is being constructed into an unaligned char array, and the DecimalFormatProperties' MeasureUnit field assigns to an int32_t field in its ctor.

      Possible fix: add alignas as follows:

      alignas(DecimalFormatProperties)
char kRawDefaultProperties[sizeof(DecimalFormatProperties)];

        Attachments

          Issue links

          is duplicated by

          Bug - A defect that causes user-visible problems or impairs the functions of the product. ICU-20039 Unaligned memory access leads to a crash

          • assess -
          • Done

          Bug - A defect that causes user-visible problems or impairs the functions of the product. ICU-20040 Unaligned memory access in DecimalFormatProperties

          • assess -
          • Done
          relates to

          Bug - A defect that causes user-visible problems or impairs the functions of the product. ICU-20049 Maintenance ticket for ICU 62

          • assess -
          • Accepted

          Bug - A defect that causes user-visible problems or impairs the functions of the product. ICU-20223 crash in unicode set during icu_62::NumberFormat::parse(icu_62::UnicodeString const&, icu_62::Formattable&, UErrorCode&)

          • major -
          • Done

          Enhancement - Under-the-hood changes with minimal user-visible behavior changes to reduce technical debt or improve performance. ICU-20048 replace uses of cmemory.h UAlignedMemory with C++11 alignas(type)

          • medium -
          • Accepted

            Activity

              People

              • Assignee:
                shane Shane Carr
                Reporter:
                shane Shane Carr
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: