Returns different OUDIDs under iPhone sim 5.1

[mabene]

I'm using OpenUDID, unmodified, to replace the deprecated Apple-provided uniqueIdentifier api.

When compiling for iOS 5.1 everything works fine, except that [OpenUDID value] returns a different value every time the application is relaunched (I'm using the 5.1 Simulator). It seems unable to store the value persistently.

Going through the code with the debugger, I can see the lines:

NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
NSString * guuid = (NSString *) [defaults objectForKey:kAppGUUID];

...returning always guid==nil.

Am I doing something wrong?

[mabene]

I was wrong; the UDID stays always the same. But it looks like the code goes through the 100-cycle loop every time the app is relaunched. Is this the intended behavior?

[jessearmand]

That's a good question. I'm curious with this loop as well. Even though this 100-cycle loop is not really a noticeable processing bottleneck.

I tried to put a break, when the availableSlotPBid is assigned:

    for (int n=0; n<kOpenUDIDRedundancySlots; n++) {
        NSString* slotPBid = [NSString stringWithFormat:@"%@%d",kOpenUDIDSlotPBPrefix,n];
#if TARGET_OS_IPHONE || TARGET_IPHONE_SIMULATOR
        UIPasteboard* slotPB = [UIPasteboard pasteboardWithName:slotPBid create:NO];
#else
        NSPasteboard* slotPB = [NSPasteboard pasteboardWithName:slotPBid];
#endif
        OpenUDIDLog(@"SlotPB name = %@",slotPBid);
        if (slotPB==nil) {
            // assign availableSlotPBid to be the first one available
            if (availableSlotPBid==nil) { 
                availableSlotPBid = slotPBid;
                break;
            }
        } else {
            NSDictionary* dict = [OpenUDID _getDictFromPasteboard:slotPB];
            NSString* oudid = [dict objectForKey:kOpenUDIDKey];
            OpenUDIDLog(@"SlotPB dict = %@",dict);
            if (oudid==nil) {
                // availableSlotPBid could inside a non null slot where no oudid can be found
                if (availableSlotPBid==nil) availableSlotPBid = slotPBid;
            } else {
                // increment the frequency of this oudid key
                int count = [[frequencyDict valueForKey:oudid] intValue];
                [frequencyDict setObject:[NSNumber numberWithInt:++count] forKey:oudid];
            }
            // if we have a match with the bundleid, then let's look if the external UIPasteboard representation marks this app as OptedOut
            NSString* bid = [dict objectForKey:kOpenUDIDBIDKey];
            if (bid!=nil && [bid isEqualToString:bundleid]) {
                myRedundancySlotPBid = slotPBid;
                optedOutDate = [dict objectForKey:kOpenUDIDOOTSKey];
                optedOut = optedOutDate!=nil;
            }
        }
    }

This should be okay, right? Since it doesn't need to keep looking for available slots when one is already found.

[ylechelle]

Well, no, you need to go through the loop to find out if there has been an opt out for that bundleid/guuid... perhaps a break after the optedOut makes sense indeed.