Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-22545

my_vsnprintf behaves not as in C standard

Details

    Description

      Some developers apparently don't like that now my_vsnprintf("%.*s") behaves not as in the C standard.

      So, we'll change it back and introduce %S to print the truncation indicator.

      All %s that print something to the end user have to be changed, for example, to %S. This includes all error messages and everything in the code too.

      Don't forget to update the documentation in the service header file.

      And we'll need some way to ensure that in the future all new printfs to the use also use %S.

      Attachments

        Issue Links

          is caused by

          Bug - A problem which impairs or prevents the functions of the product. MDEV-20604 Duplicate key value is silently truncated to 64 characters in print_keydup_error

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed
          relates to

          Task - A task that needs to be done. MDEV-21978 make my_vsnprintf to use gcc-compatible format extensions

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-22147 main.mysqldump fails with wrong result (truncated output)

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            serg Sergei Golubchik created issue -
            serg Sergei Golubchik made changes -
            Field Original Value New Value
            Description Some developers apparently don't like that now {{my_vsnprintf("%.*s")}} behaves not as in the C standard.

            So, we'll change it back and introduce {{%S}} to print the truncation indicator.

            All {{%s}} that print something to the end user have to be changed to {{%S}}. This includes all error messages and everything in the code too.

            And we'll need some way to ensure that in the future all new printfs to the use also use {{%S}}.             		Some developers apparently don't like that now {{my_vsnprintf("%.*s")}} behaves not as in the C standard.

            So, we'll change it back and introduce {{%S}} to print the truncation indicator.

            All {{%s}} that print something to the end user have to be changed to {{%S}}. This includes all error messages and everything in the code too.

            Don't forget to update the documentation in the service header file.

            And we'll need some way to ensure that in the future all new printfs to the use also use {{%S}}.
            serg Sergei Golubchik made changes -
            Description Some developers apparently don't like that now {{my_vsnprintf("%.*s")}} behaves not as in the C standard.

            So, we'll change it back and introduce {{%S}} to print the truncation indicator.

            All {{%s}} that print something to the end user have to be changed to {{%S}}. This includes all error messages and everything in the code too.

            Don't forget to update the documentation in the service header file.

            And we'll need some way to ensure that in the future all new printfs to the use also use {{%S}}.             		Some developers apparently don't like that now {{my_vsnprintf("%.*s")}} behaves not as in the C standard.

            So, we'll change it back and introduce {{%S}} to print the truncation indicator.

            All {{%s}} that print something to the end user have to be changed, for example, to {{%S}}. This includes all error messages and everything in the code too.

            Don't forget to update the documentation in the service header file.

            And we'll need some way to ensure that in the future all new printfs to the use also use {{%S}}.
            serg Sergei Golubchik made changes -
            marko Marko Mäkelä made changes -
            marko Marko Mäkelä made changes -
            Affects Version/s 10.5.3 [ 24263 ]
            Affects Version/s 10.4.13 [ 24223 ]
            Affects Version/s 10.3.23 [ 24222 ]
            Affects Version/s 10.2.32 [ 24221 ]
            Affects Version/s 10.2 [ 14601 ]
            Affects Version/s 10.3 [ 22126 ]
            Affects Version/s 10.4 [ 22408 ]
            Affects Version/s 10.5 [ 23123 ]
            sanja Oleksandr Byelkin made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            sanja Oleksandr Byelkin made changes -
            Priority Major [ 3 ] Blocker [ 1 ]
            sanja Oleksandr Byelkin made changes -
            Assignee Oleksandr Byelkin [ sanja ] Sergei Golubchik [ serg ]
            Status In Progress [ 3 ] In Review [ 10002 ]
            serg Sergei Golubchik made changes -
            serg Sergei Golubchik made changes -
            Assignee Sergei Golubchik [ serg ] Oleksandr Byelkin [ sanja ]
            Status In Review [ 10002 ] Stalled [ 10000 ]
            sanja Oleksandr Byelkin made changes -
            Fix Version/s 10.3.24 [ 24306 ]
            Fix Version/s 10.4.14 [ 24305 ]
            Fix Version/s 10.2.34 [ 24505 ]
            Fix Version/s 10.2 [ 14601 ]
            Fix Version/s 10.3 [ 22126 ]
            Fix Version/s 10.4 [ 22408 ]
            Fix Version/s 10.5 [ 23123 ]
            Resolution Fixed [ 1 ]
            Status Stalled [ 10000 ] Closed [ 6 ]
            sanja Oleksandr Byelkin made changes -
            Fix Version/s 10.5.4 [ 24264 ]
            ralf.gebhardt Ralf Gebhardt made changes -
            Fix Version/s 10.2.35 [ 25022 ]
            ralf.gebhardt Ralf Gebhardt made changes -
            Fix Version/s 10.2.34 [ 24505 ]
            serg Sergei Golubchik made changes -
            Workflow MariaDB v3 [ 108626 ] MariaDB v4 [ 157762 ]

            People

              sanja Oleksandr Byelkin
              serg Sergei Golubchik
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.