-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3335/
-----------------------------------------------------------
(Updated March 24, 2014, 10:07 a.m.)
Review request for Asterisk Developers.
Changes
-------
The check for text column moved to an inline function. Minor cleanups.
Bugs: ASTERISK-23459
https://issues.asterisk.org/jira/browse/ASTERISK-23459
Repository: Asterisk
Description
-------
This patch fixes setting nullable integer columns to NULL instead of an empty
string, which fails for PostgreSQL, for example.
The current code is supposed to do so, but the check is broken. The patch also
allows the first column in the list to be a nullable integer.
Also, the check for existence of a mandatory column checked for the first
column in the list instead of the key field lookup column.
Diffs (updated)
-----
http://svn.asterisk.org/svn/asterisk/trunk/res/res_config_odbc.c 410980
Diff: https://reviewboard.asterisk.org/r/3335/diff/
Testing
-------
Tested by me. Use case scenario: Asterisk + res_odbc + PostgreSQL backend, SIP
realtime peers + regs.
When a 'port' column in SIP regs (I assume this also applies when using
sippeers only) is a nullable integer,
Asterisk tries to write an empty string here during SIP endpoint deregistration.
Thanks,
zvision
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
