ISO14443aTarget marshalling problem? about nfc

[clausecker]

Sounds like a bug in my code to me. Let me check please.

[clausecker]

unsafe_p := (unsafe.Pointer(targets[0].Marshall()))
defer C.free(unsafe_p)
mtarget := (*nfc.ISO14443aTarget)(unsafe_p)

Casting an unsafe.Pointer you get from Marshall() to one of the target types can't work. The pointer you get from Marshall() points to a C type, whereas the target types are entirely different types. As per documentation for Target.Marshall():

Marshall() returns a pointer to an nfc_target allocated with C.malloc() that contains the same data as the Target. Don't forget to C.free() the result of Marshall() afterwards. A runtime panic may occur if any slice referenced by a Target has been made larger than the maximum length mentioned in the respective comments.

Where nfc_target refers to the C type nfc_target, not the Go type. If you want to convert a Target into an object of concrete type, use a type assertion like this (replacing the code above):

mtarget, ok := targets[0].(nfc.ISO14443aTarget)
if !ok {
    // targets[0] is apparently not an ISO 14443a target. Do something here...
}

Please tell me if this solved your problem and then close the issue.

[messinm]

Thanks for the quick response.

I tried your suggestion, but it won't compile. See snippet and error message below. Seems that nfc.ISO14443aTarget should implement nfc.Target, but doesn't.

                            //unsafe_p := (unsafe.Pointer(targets[0].Marshall()))
                            //defer C.free(unsafe_p)
                            //mtarget := (*nfc.ISO14443aTarget)(unsafe_p) 

                            mtarget, ok := targets[0].(nfc.ISO14443aTarget)
                            if !ok {
                               fmt.Printf("Not ISO14443aTarget!")
                            } else {
            fmt.Printf("Atqa =%v\n",mtarget.Atqa)
            fmt.Printf("Sak =%v\n",mtarget.Sak) 
                    fmt.Printf("UIDLen =%v\n",mtarget.UIDLen)  
                    fmt.Printf("UID =%v\n",mtarget.UID) 
                            }

//does not compile:
//../src/messinm2/testnfc/testnfc.go:81: impossible type assertion:
// nfc.ISO14443aTarget does not implement nfc.Target (Marshall method has pointer receiver)

[clausecker]

Sorry. Cast to *nfc.ISO14443aTarget, not nfc.ISO14443aTarget. I don't know why I forgot that star.

[messinm]

That works! Thanks, and thanks very much for the lib.

[clausecker]

It was a pleasure to me.