Reporting crashed, when maxInterval is 0! Cyclic reboot.

Slacky

Hi.




Bug in function reportAttrs()

Slacky

Sorry. Cyclic reboot, if watch_dog enabled. If disabled, no return from reportAttrs().

wes58

Slacky 发表于 2025-6-12 03:25
Sorry. Cyclic reboot, if watch_dog enabled. If disabled, no return from reportAttrs().

Did you try to find in the source code where the problem is?
You say that there is "no return from reportAttrs()"
If you look in the source code, the only way it stays in the "do while" loop is when again = 1.
And it is set twice in this function.

1. }else if((clusterID != pEntry->clusterID) || (profileID != pEntry->profileID) || (endpoint != pEntry->endPoint)){
   
2.            again = 1;
   
3.            continue;
   
4. }

复制代码

or

1. if(report.numAttr >= 2){
   
2.          again = 1;
   
3.          break;
   
4. }
   

复制代码

It is easy for you to check it, because you have this problem. You can add some variables in the code and see what is happening or use some LEDs to have visual indication etc.
Otherwise post all your project code so someone can check it.

That is my advice to you, otherwise you may be waiting for a long time.

Slacky

The ZCL says what to do if maxInterval is 0.




Adding a reportableChange check to the reportAttrs(void) function fixes the issue.

                for(u8 i = 0; i < ZCL_REPORTING_TABLE_NUM; i++){

                        pEntry = &reportingTab.reportCfgInfo;



                        if(pEntry->used && (pEntry->maxInterval != 0xFFFF) &&

                           zb_bindingTblSearched(pEntry->clusterID, pEntry->endPoint)){

                                pAttrEntry = zcl_findAttribute(pEntry->endPoint, pEntry->clusterID, pEntry->attrID);

                                if(pAttrEntry){

                                        bool valid = 0;

                                        u8 dataLen = zcl_getAttrSize(pAttrEntry->type, pAttrEntry->data);



                                        if(!pEntry->maxIntCnt){

                        if((!zcl_analogDataType(pAttrEntry->type) && memcmp(pEntry->prevData, pAttrEntry->data, dataLen)) ||

                            (zcl_analogDataType(pAttrEntry->type) && reportableChangeValueChk(pAttrEntry->type, pAttrEntry->data,

                                                                                              pEntry->prevData, pEntry->reportableChange))){

                            valid = 1;

                        }

                                        }else if(!pEntry->minIntCnt){

                                                if((!zcl_analogDataType(pAttrEntry->type) && memcmp(pEntry->prevData, pAttrEntry->data, dataLen)) ||

                                                        (zcl_analogDataType(pAttrEntry->type) && reportableChangeValueChk(pAttrEntry->type, pAttrEntry->data,

                                                                                                                                                                                          pEntry->prevData, pEntry->reportableChange))){

                                                        valid = 1;

                                                }else{

                                                        pEntry->minIntCnt = pEntry->minInterval;

                                                }

                                        }

Slacky

It seems that this is correct.

            if (pEntry->used && (pEntry->maxInterval != 0xFFFF) && zb_bindingTblSearched(pEntry->clusterID, pEntry->endPoint)) {

                pAttrEntry = zcl_findAttribute(pEntry->endPoint, pEntry->clusterID, pEntry->attrID);

                if (pAttrEntry) {

                    bool valid = 0;

                    u8 dataLen = zcl_getAttrSize(pAttrEntry->type, pAttrEntry->data);



                    if (!pEntry->maxIntCnt) {

                        if (!pEntry->maxInterval) {

                            if ((!zcl_analogDataType(pAttrEntry->type) && memcmp(pEntry->prevData, pAttrEntry->data, dataLen)) ||

                                 (zcl_analogDataType(pAttrEntry->type) && reportableChangeValueChk(pAttrEntry->type,

                                                                                                   pAttrEntry->data,

                                                                                                   pEntry->prevData,

                                                                                                   pEntry->reportableChange))) {

                                valid = 1;

                            }

                        } else {

                            valid = 1;

                        }

                    } else if (!pEntry->minIntCnt) {

                        if ((!zcl_analogDataType(pAttrEntry->type) && memcmp(pEntry->prevData, pAttrEntry->data, dataLen)) ||

                             (zcl_analogDataType(pAttrEntry->type) && reportableChangeValueChk(pAttrEntry->type,

                                                                                               pAttrEntry->data,

                                                                                               pEntry->prevData,

                                                                                               pEntry->reportableChange))) {

                            valid = 1;

                        } else {

                            pEntry->minIntCnt = pEntry->minInterval;

                        }

                    }



                    if (valid) {

TL_YANGBIN

Slacky 发表于 2025-6-14 00:42
It seems that this is correct.

Thank you for your feedback. I will synchronize this issue to the next version.

we_9568922770

What problems would arise with the following modification?

            if (pEntry->used && (pEntry->maxInterval != 0xFFFF) &&
                zb_bindingTblSearched(pEntry->clusterID, pEntry->endPoint)) {
                pAttrEntry = zcl_findAttribute(pEntry->endPoint, pEntry->clusterID, pEntry->attrID);
                if (pAttrEntry) {
                    bool valid = 0;
                    u8 dataLen = zcl_getAttrSize(pAttrEntry->type, pAttrEntry->data);

                    dataLen = (dataLen > REPORTABLE_CHANGE_MAX_ANALOG_SIZE) ? (REPORTABLE_CHANGE_MAX_ANALOG_SIZE) : (dataLen);
                    if (!pEntry->maxIntCnt && pEntry->maxInterval) {
                        valid = 1;
                    } else if (!pEntry->minIntCnt) {
                        if ((!zcl_analogDataType(pAttrEntry->type) && memcmp(pEntry->prevData, pAttrEntry->data, dataLen)) ||
                            (zcl_analogDataType(pAttrEntry->type) && reportableChangeValueChk(pAttrEntry->type, pAttrEntry->data,
                                                                                              pEntry->prevData, pEntry->reportableChange))) {
                            valid = 1;
                        } else {
                            pEntry->minIntCnt = pEntry->minInterval;
                        }
                    }

                    if (valid) {