[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sr-dev
Subject:    [OpenSER-Devel] [ openser-Bugs-1954586 ] postgres: bogus error on
From:       noreply () sourceforge ! net (SourceForge ! net)
Date:       2008-04-30 14:08:33
Message-ID: E1JrCz3-0004Oo-DB () sc8-sf-web23 ! sourceforge ! net
[Download RAW message or body]

Bugs item #1954586, was opened at 2008-04-30 00:38
Message generated for change (Comment added) made by henningw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1954586&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver 1.3.x
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ovidiu Sas (osas)
> Assigned to: Henning Westerholt (henningw)
Summary: postgres: bogus error on empty avp_db_query

Initial Comment:
If a query is performed on a postgres db using the avp_db_query() and the query \
returns no rows, the following bogus error is printed on the logs: \
ERROR:postgres:pg_convert_rows: no rows requested from the query If the query is \
returning something, then no errors are printed out.


Regards,
Ovidiu Sas

----------------------------------------------------------------------

> Comment By: Henning Westerholt (henningw)
Date: 2008-04-30 14:08

Message:
Logged In: YES 
user_id=337916
Originator: NO

Hi Ovidiu,

this error message is in fact emitted for every empty query. I reviewed
the code of the function in question, and found some additional issues.
I'll take care of this.

Cheers,

Henning

----------------------------------------------------------------------

Comment By: Ovidiu Sas (osas)
Date: 2008-04-30 00:45

Message:
Logged In: YES 
user_id=1395524
Originator: YES

dbase.c:pg_fetch_result() checks the number of rows before calling
pg_convert_rows()
db_res.c:pg_convert_result() doesn't check if the number of rows is 0
(only <0 check is performed) before calling pg_convert_rows()

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1954586&group_id=139143


[prev in list] [next in list] [prev in thread] [next in thread]