Handling of negative timeout values in API

Brought to you by: anders-w, gary_lee, hansnordeback, jonasarndt, and 4 others

#738 Handling of negative timeout values in API

Milestone: 4.6.FC

Status: fixed

Owner: Mathi Naickan

Labels: None

Type: defect

Component: osaf

Part: lib

Version: 4.4.M0

Priority: minor

Blocker:

Updated: 2015-03-13

Created: 2014-01-22

Creator: Sirisha Alla

Private: No

The issue is seen on changeset 4733 | patches for #220.

saClmClusterNodeGet_4 API is invoked with -3 as input for the timeout parameter. ERR_TIMEOUT is returned where in the closest return value to be returned would be INVALID_PARAM according to the specification.

Related

Tickets: ~~#738~~

Discussion

Sirisha Alla - 2014-01-22

summary: INVALID_PARAM nees to be returned when a timeout in negative is passed for ClusterNodeGet_4 --> INVALID_PARAM needs to be returned when a timeout in negative is passed for ClusterNodeGet_4
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-01-23

status: unassigned --> assigned

assigned_to: Mathi Naickan

Milestone: future --> 4.4.RC1
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-01-27

summary: INVALID_PARAM needs to be returned when a timeout in negative is passed for ClusterNodeGet_4 --> Handling of negative timeout values in API

Type: defect --> discussion

Component: clm --> osaf

Part: - --> doc
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-01-27

The headline of this ticket is invalid in the general sense.
INVALID_PARM for a negative value would not be apt given that SaTimeT is signed.
However, a clarification can be considered in the OpenSAF PR as a general guideline for the enduser, to explain how OpenSAF services shall treat a -1. Need discussion on this.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-02-05

Milestone: 4.4.RC1 --> 4.5.FC
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nagendra Kumar - 2014-02-06

A duplicate ticket(https://sourceforge.net/p/opensaf/tickets/768/) is linked to it.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anders Widell - 2014-08-15

Milestone: 4.5.FC --> 4.5.0
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-09-15

In preparation for 4.5.RC1.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2014-09-15

Milestone: 4.5.0 --> 4.6.FC
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2015-02-23

status: assigned --> accepted

Type: discussion --> enhancement

Part: doc --> lib
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2015-03-11

status: accepted --> review

Type: enhancement --> defect
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2015-03-11

Changing this to defect because the current behaviour seems to have been introduced sometime in 4.3.

https://sourceforge.net/p/opensaf/mailman/opensaf-devel/thread/patchbomb.1426063995%40ubuntu/#msg33583492

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2015-03-13

changeset: 6353:0f6bdf7f489c
tag: tip
user: Mathivanan N.P.mathi.naickan@oracle.com
date: Fri Mar 13 15:46:09 2015 +0530
summary: clm: return SA_AIS_ERR_INVALID_PARAM for negative timeout values [#738]

Related

Tickets: ~~#738~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Mathi Naickan - 2015-03-13

status: review --> fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.