Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BACnet-Reject-PDU and BACnet-Abort-PDU to be supported #14

Open
duffy-ocraven opened this issue Aug 26, 2020 · 1 comment
Open

BACnet-Reject-PDU and BACnet-Abort-PDU to be supported #14

duffy-ocraven opened this issue Aug 26, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@duffy-ocraven
Copy link

duffy-ocraven commented Aug 26, 2020

BACnet-Reject-PDU ::= SEQUENCE {
 	pdu-type 		Unsigned (0..15), -- high four bits, 0x60 for this PDU type
 	reserved 			Unsigned (0..15), -- low four bits, always 0 for this PDU type
 	original-invokeID 	Unsigned (0..255), -- matching what was in the ConfirmedRequest to which this is response
 	reject-reason 		One octet containing the BACnetRejectReason

and the BACnetRejectReason enumerated is:{
other (0),
buffer-overflow (1),
inconsistent-parameters (2),
invalid-parameter-data-type (3),
invalid-tag (4),
missing-required-parameter (5),
parameter-out-of-range (6),
too-many-arguments (7),
undefined-enumeration (8),
unrecognized-service (9),

BACnet-Abort-PDU ::= SEQUENCE {
	pdu-type		Unsigned (0..15), -- high 4 bits, thus 0x70 or 0x71 for this PDU type
	server			Unsigned (0..15) -- BOOLEAN 0 == FALSE, 1 == TRUE,
	original-invoke-id	Unsigned (0..255), -- matching what was in the ConfirmedRequest to which this is response
	abort-reason		One octet containing the BACnetAbortReason

and the BACnetAbortReason enumerated is: {
other (0),
buffer-overflow (1),
invalid-apdu-in-this-state (2),
preempted-by-higher-priority-task (3),
segmentation-not-supported (4),
security-error (5),
insufficient-security (6),
window-size-out-of-range (7),
application-exceeded-reply-time (8),
out-of-resources (9),
tsm-timeout (10),
apdu-too-long (11),

Note that unlike the BACnet-Error-PDU, in these two the ServiceChoice is not present.

@NothinRandom NothinRandom self-assigned this Aug 26, 2020
@NothinRandom NothinRandom added the enhancement New feature or request label Aug 31, 2020
@NothinRandom
Copy link
Contributor

@duffy-corelight, latest update and enumeration addresses this issue. Like the others, I'll let the customer close the issue if deemed as satisfied.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants