Mailing List Archive

wrong variable name in pgsql
Hi,

In get_pgsql_param() of pgsql, the local variable "config_file" is defined,
but not used. OTOH, the variable "config" is used but not defined as
a local variable. Obviously we should correct the variable name from
"config_file" to "config". Patch attached.

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
Re: wrong variable name in pgsql [ In reply to ]
Hi,

On Thu, Oct 27, 2011 at 11:08:07PM +0900, Fujii Masao wrote:
> Hi,
>
> In get_pgsql_param() of pgsql, the local variable "config_file" is defined,
> but not used. OTOH, the variable "config" is used but not defined as
> a local variable. Obviously we should correct the variable name from
> "config_file" to "config". Patch attached.

Applied. Many thanks for the patch!

Cheers,

Dejan

> Regards,
>
> --
> Fujii Masao
> NIPPON TELEGRAPH AND TELEPHONE CORPORATION
> NTT Open Source Software Center

> diff --git a/heartbeat/pgsql b/heartbeat/pgsql
> index ba80790..a55cf66 100755
> --- a/heartbeat/pgsql
> +++ b/heartbeat/pgsql
> @@ -20,7 +20,7 @@
> # Get PostgreSQL Configuration parameter
> #
> get_pgsql_param() {
> - local config_file
> + local config
> local param_name
>
> param_name=$1

> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/