Opened 3 years ago

#12190 new defect

Pyptlib does not join args or optargs correctly in ClientTransportPlugin.reportMethodSuccess

Reported by: RushingWookie Owned by: asn
Priority: Low Milestone:
Component: Obfuscation/Pluggable transport Version:
Severity: Keywords: pyptlib, ClientTransportPlugin, ReportMethodSuccess
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

In ClientTransportPlugin.reportMethodSuccess, the code for joining the params args and optargs is wrong. The args and optargs params are defined as strings when they should be lists of strings.
Current code

        if args and len(args) > 0:
          methodLine = methodLine + ' ARGS=' + args.join(',')
        if optArgs and len(optArgs) > 0:
          methodLine = methodLine + ' OPT-ARGS=' + args.join(',')

Fixed Code

        if args and len(args) > 0:
          methodLine = methodLine + ' ARGS=' + ','.join(args)
        if optArgs and len(optArgs) > 0:
          methodLine = methodLine + ' OPT-ARGS=' + ','.join(optArgs)

Also the documentation for the types of args and optArgs should be changed to lists.

Child Tickets

Change History (0)

Note: See TracTickets for help on using tickets.