Camel
  1. Camel
  2. CAMEL-4573

camel-bindy BindyFixedLength parser is not Thread Safe while marshalling.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.1
    • Fix Version/s: 2.8.3, 2.9.0
    • Component/s: camel-bindy
    • Labels:
    • Environment:

      linux, windows

    • Patch Info:
      Patch Available
    • Estimated Complexity:
      Advanced

      Description

      Camel Fixed Length Parser. Not Threadsafe, as instance variable private Map<Integer, List<String>> results; is being used while marshalling/unbinding. Inconsistencies in multithreaded environment causing data from different threads being used.

      Very similar to BUG 4311 ( for KeyValuePair Parser)

        Activity

        surya created issue -
        Hide
        surya added a comment -

        Fixed the Code, Changed instance variable to local one so that it is thread safe.

        Show
        surya added a comment - Fixed the Code, Changed instance variable to local one so that it is thread safe.
        surya made changes -
        Field Original Value New Value
        Attachment camel_bindy_patch_4573 [ 12500279 ]
        Claus Ibsen <davsclaus@apache.org> committed 608b3cbe0becbbbf27b1a965d91c6b2629613776 (1 file)
        Reviews: none

        CAMEL-4573: Fixed thread safety issue in bindy fixed length marshaller. Thanks to Surya for the patch.
        git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@1187671 13f79535-47bb-0310-9956-ffa450edef68

        Claus Ibsen committed 1187671 (1 file)
        Reviews: none

        CAMEL-4573: Fixed thread safety issue in bindy fixed length marshaller. Thanks to Surya for the patch.

        Claus Ibsen <davsclaus@apache.org> committed 0366c09c039a9957a92fec594e01dc647bae9519 (1 file)
        Reviews: none

        CAMEL-4573: Fixed thread safety issue in bindy CSV marshaller.
        git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@1187672 13f79535-47bb-0310-9956-ffa450edef68

        Claus Ibsen committed 1187672 (1 file)
        Hide
        Claus Ibsen added a comment -

        Thanks for the patch. I fixed the same issue in the CSV as well. So all three is now fixed.

        Show
        Claus Ibsen added a comment - Thanks for the patch. I fixed the same issue in the CSV as well. So all three is now fixed.
        Claus Ibsen made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.8.3 [ 12318649 ]
        Resolution Fixed [ 1 ]
        Claus Ibsen made changes -
        Assignee Claus Ibsen [ davsclaus ]
        Claus Ibsen <davsclaus@apache.org> committed 9240419b1a6490ff8a5c5d400da89f4f78541bec (2 files)
        J. Daniel Kulp <dkulp@apache.org> committed aea22bb5689f075ad283262f7d7a99aa2cf255dc (0 files)
        Reviews: none

        Recorded merge of revisions 1187671-1187672 via svnmerge from https://svn.apache.org/repos/asf/camel/trunk
        ........
          r1187671 | davsclaus | 2011-10-22 04:34:45 -0400 (Sat, 22 Oct 2011) | 1 line
          
          CAMEL-4573: Fixed thread safety issue in bindy fixed length marshaller. Thanks to Surya for the patch.
        ........
          r1187672 | davsclaus | 2011-10-22 04:39:40 -0400 (Sat, 22 Oct 2011) | 1 line
          
          CAMEL-4573: Fixed thread safety issue in bindy CSV marshaller.
        ........


        git-svn-id: https://svn.apache.org/repos/asf/camel/branches/camel-2.8.x@1201837 13f79535-47bb-0310-9956-ffa450edef68

        Daniel Kulp committed 1201837 (1 file)
        Reviews: none

        Recorded merge of revisions 1187671-1187672 via svnmerge from
        https://svn.apache.org/repos/asf/camel/trunk

        ........
          r1187671 | davsclaus | 2011-10-22 04:34:45 -0400 (Sat, 22 Oct 2011) | 1 line
          
          CAMEL-4573: Fixed thread safety issue in bindy fixed length marshaller. Thanks to Surya for the patch.
        ........
          r1187672 | davsclaus | 2011-10-22 04:39:40 -0400 (Sat, 22 Oct 2011) | 1 line
          
          CAMEL-4573: Fixed thread safety issue in bindy CSV marshaller.
        ........

          People

          • Assignee:
            Claus Ibsen
            Reporter:
            surya
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development