Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-1264

TSocketClient is queried by run loop after deallocation in Cocoa

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 0.7, 0.9
    • 0.9.1
    • Cocoa - Library
    • None
    • iPad, iOS SDK 4.3, iOS SDK 5.1

    • Patch Available

    Description

      When using TSocketClient in a Cocoa client and correctly managing the memory:

      Server.m
      - (id)initWithServer:(NSString*)server port:(NSInteger)port
      {
          self = [super init];
          if (self) {
              TSocketClient *transport = [[TSocketClient alloc] initWithHostname:server 
                                                             port:port];
              TBinaryProtocol *protocol = [[TBinaryProtocolFactory sharedFactory] newProtocolOnTransport:transport];
              Client *client = [[Client alloc] initWithProtocol:protocol];
      
      // self.client is a retained property
              self.client = client;
              [client release];
              [protocol release];
              [transport release];
          }
          
          return self;
      }
      
      
      - (void)dealloc {
          self.client = nil;
          [super dealloc];
      }
      

      (maybe this should be fixed in the cocoa example, memory management is erroneous there)

      After successfully releasing the object with the code above. One gets a EXC_BAD_ACCESS signal because the runloop send a responseToSelector: message to our object. With the selector stream:handleEvent:

      This all leads to TSocketClient which opens two NSStream objects and delegates it self to both, afterwards both Streams are scheduled in the runloop, which should actually notify the delegate when something happens to the stream like data is available, this obviously results in the runloop asking TSocketClient for previously described selector. However this mechanism is not used, the TNSStreamTransport class is subclassed and responsible for read/write.

      So why are the Streams schedules in the run loop in the first place?
      Why are the streams not closed and released when the object gets deallocated?

      I could fix this behavior by implementing a dealloc method in TSocketClient and making the Streams member variables

      TSocketClient.h
      @interface TSocketClient : TNSStreamTransport {
          NSInputStream * inputStream;
      	NSOutputStream * outputStream;
      }
      
      TSocketClient.m
      -(void)dealloc {
          [inputStream close];
          [outputStream close];
          [inputStream release];
          [outputStream release];
          [super dealloc];
      }
      

      However I still don't know if that is the right way.
      Suggestions?

      Edit:

      The bug is still present in 0.9 I attached a TSocketClient.m that fixes this Problem, the TSocketClient also keeps track of the streams and upon deallocation it closes the stream, removes it from the runloop and releases it (if arc is not enabled). This also removes a bug in non-arc environment that where the streams have a retaincount of +1 after deallocation such that the bug described here won't occur and the streams leak memory

      Attachments

        1. TSocketClient.m
          3 kB
          Jan Rüth
        2. THRIFT-1264.diff
          1 kB
          Jan Rüth

        Activity

          People

            Unassigned Unassigned
            eichhoernchen Jan Rüth
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: