Crash with Door Lock device

[tlaukkan]

Crash with Door Lock device

[presslab-us]

I have seen something like this while developing a ZigBee device. I believe the problem is that zigbee4java does not check if the descriptor length is zero and tries to parse it. I think I can come up with a patch, give me a few hours.

[presslab-us]

Yes, I see the problem in the code. By the way, what lock are you working with?

[smulikHakipod]

Actually I am using the sample lock from TI HA examples. And you are probably right about the length, when I debug the crash I saw that the packet length is zero.

[presslab-us]

Well, there you go. If you look at sampleDoorLock_TestEp in the sample code, you can see there is no simple descriptor. This is what triggers the bug in zigbee4java. You can just remove the afRegister for this endpoint, you don't need it.

What sort of lock are you working on? I plan to make some kind of DIY ZigBee servo actuator for my deadbolt.

[presslab-us]

This should fix it, please test and report back. a9654de

[smulikHakipod]

Thanks for your help!!
I will report back ASAP.
About the door lock, the manufacture of doors here uses some unique mechanism that isn't compatible with any smart lock I found. I draw using OpenSCAD a plastic that wraps the around the key and I want the CC2530 to control it.

http://imgur.com/a/dlIDZ

[presslab-us]

That is a strong looking gear motor, and your part looks very nice. Here are some of my DIY ZigBee efforts: https://groups.google.com/forum/#!topic/zigbee4java/nREaHiQW42s

[smulikHakipod]

Your solution fixed the bug, many thanks!
As for the door lock, it looks like the zigbee4java does not implement the door lock device, so ZigbeeApi matches my door lock as Dimmable Light (working on fixing it now by adding DoorLockDevice?).

[presslab-us]

Yes I have seen this too. If the device ID is not supported it seemingly picks a different one at random. Your device ID is 0x000A, yes?

You will need to add a door lock device 0x000A, the door lock cluster 0x0101, the cluster attributes 0-6, and the two lock/unlock cluster commands.

[smulikHakipod]

I added the changes to my fork:
master...smulikHakipod:master
For now, really basic lock command and lock state.
Are the names and structure OK? Did I miss anything? Should I make a pull request?
thanks

[tlaukkan]

Hi

When you are happy with your code please make a pull request and we will review and merge it to master.

Best regards,
Tommi

[tlaukkan]

Nice work by the way and very useful.

[tlaukkan]

Thank you for the fix @presslab-us. 👍