mirror of
https://github.com/postgres/postgres.git
synced 2025-09-03 15:22:11 +03:00
Fix up warning and error messages to use single-quotes around strings.
This commit is contained in:
@@ -7,7 +7,7 @@
|
|||||||
*
|
*
|
||||||
*
|
*
|
||||||
* IDENTIFICATION
|
* IDENTIFICATION
|
||||||
* $Header: /cvsroot/pgsql/src/backend/commands/dbcommands.c,v 1.24 1998/09/01 04:27:51 momjian Exp $
|
* $Header: /cvsroot/pgsql/src/backend/commands/dbcommands.c,v 1.25 1998/10/05 02:49:36 thomas Exp $
|
||||||
*
|
*
|
||||||
*-------------------------------------------------------------------------
|
*-------------------------------------------------------------------------
|
||||||
*/
|
*/
|
||||||
@@ -77,7 +77,7 @@ createdb(char *dbname, char *dbpath, int encoding)
|
|||||||
" in the server", loc);
|
" in the server", loc);
|
||||||
|
|
||||||
if (mkdir(lp, S_IRWXU) != 0)
|
if (mkdir(lp, S_IRWXU) != 0)
|
||||||
elog(ERROR, "Unable to create database directory %s", lp);
|
elog(ERROR, "Unable to create database directory '%s'", lp);
|
||||||
|
|
||||||
sprintf(buf, "%s %s%cbase%ctemplate1%c* %s",
|
sprintf(buf, "%s %s%cbase%ctemplate1%c* %s",
|
||||||
COPY_CMD, DataDir, SEP_CHAR, SEP_CHAR, SEP_CHAR, lp);
|
COPY_CMD, DataDir, SEP_CHAR, SEP_CHAR, SEP_CHAR, lp);
|
||||||
@@ -85,12 +85,12 @@ createdb(char *dbname, char *dbpath, int encoding)
|
|||||||
|
|
||||||
#if FALSE
|
#if FALSE
|
||||||
sprintf(buf, "insert into pg_database (datname, datdba, datpath) \
|
sprintf(buf, "insert into pg_database (datname, datdba, datpath) \
|
||||||
values (\'%s\'::name, \'%d\'::oid, \'%s\'::text);",
|
values ('%s'::name, '%d'::oid, '%s'::text);",
|
||||||
dbname, user_id, dbname);
|
dbname, user_id, dbname);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
sprintf(buf, "insert into pg_database (datname, datdba, encoding, datpath)"
|
sprintf(buf, "insert into pg_database (datname, datdba, encoding, datpath)"
|
||||||
" values (\'%s\', \'%d\', \'%d\', \'%s\');", dbname, user_id, encoding, loc);
|
" values ('%s', '%d', '%d', '%s');", dbname, user_id, encoding, loc);
|
||||||
|
|
||||||
|
|
||||||
pg_exec_query(buf);
|
pg_exec_query(buf);
|
||||||
@@ -112,7 +112,7 @@ destroydb(char *dbname)
|
|||||||
check_permissions("destroydb", dbpath, dbname, &db_id, &user_id);
|
check_permissions("destroydb", dbpath, dbname, &db_id, &user_id);
|
||||||
|
|
||||||
if (!OidIsValid(db_id))
|
if (!OidIsValid(db_id))
|
||||||
elog(FATAL, "impossible: pg_database instance with invalid OID.");
|
elog(FATAL, "pg_database instance has an invalid OID");
|
||||||
|
|
||||||
/* stop the vacuum daemon */
|
/* stop the vacuum daemon */
|
||||||
stop_vacuum(dbpath, dbname);
|
stop_vacuum(dbpath, dbname);
|
||||||
@@ -156,7 +156,7 @@ get_pg_dbtup(char *command, char *dbname, Relation dbrel)
|
|||||||
|
|
||||||
scan = heap_beginscan(dbrel, 0, SnapshotNow, 1, &scanKey);
|
scan = heap_beginscan(dbrel, 0, SnapshotNow, 1, &scanKey);
|
||||||
if (!HeapScanIsValid(scan))
|
if (!HeapScanIsValid(scan))
|
||||||
elog(ERROR, "%s: cannot begin scan of pg_database.", command);
|
elog(ERROR, "%s: cannot begin scan of pg_database", command);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* since we want to return the tuple out of this proc, and we're going
|
* since we want to return the tuple out of this proc, and we're going
|
||||||
@@ -211,17 +211,17 @@ check_permissions(char *command,
|
|||||||
/* Check to make sure user has permission to use createdb */
|
/* Check to make sure user has permission to use createdb */
|
||||||
if (!use_createdb)
|
if (!use_createdb)
|
||||||
{
|
{
|
||||||
elog(ERROR, "user \"%s\" is not allowed to create/destroy databases",
|
elog(ERROR, "user '%s' is not allowed to create/destroy databases",
|
||||||
userName);
|
userName);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Make sure we are not mucking with the template database */
|
/* Make sure we are not mucking with the template database */
|
||||||
if (!strcmp(dbname, "template1"))
|
if (!strcmp(dbname, "template1"))
|
||||||
elog(ERROR, "%s cannot be executed on the template database.", command);
|
elog(ERROR, "%s: cannot be executed on the template database", command);
|
||||||
|
|
||||||
/* Check to make sure database is not the currently open database */
|
/* Check to make sure database is not the currently open database */
|
||||||
if (!strcmp(dbname, DatabaseName))
|
if (!strcmp(dbname, DatabaseName))
|
||||||
elog(ERROR, "%s cannot be executed on an open database", command);
|
elog(ERROR, "%s: cannot be executed on an open database", command);
|
||||||
|
|
||||||
/* Check to make sure database is owned by this user */
|
/* Check to make sure database is owned by this user */
|
||||||
|
|
||||||
@@ -272,20 +272,20 @@ check_permissions(char *command,
|
|||||||
if (dbfound && !strcmp(command, "createdb"))
|
if (dbfound && !strcmp(command, "createdb"))
|
||||||
{
|
{
|
||||||
|
|
||||||
elog(ERROR, "createdb: database %s already exists.", dbname);
|
elog(ERROR, "createdb: database '%s' already exists", dbname);
|
||||||
|
|
||||||
}
|
}
|
||||||
else if (!dbfound && !strcmp(command, "destroydb"))
|
else if (!dbfound && !strcmp(command, "destroydb"))
|
||||||
{
|
{
|
||||||
|
|
||||||
elog(ERROR, "destroydb: database %s does not exist.", dbname);
|
elog(ERROR, "destroydb: database '%s' does not exist", dbname);
|
||||||
|
|
||||||
}
|
}
|
||||||
else if (dbfound && !strcmp(command, "destroydb")
|
else if (dbfound && !strcmp(command, "destroydb")
|
||||||
&& dbowner != *userIdP && use_super == false)
|
&& dbowner != *userIdP && use_super == false)
|
||||||
{
|
{
|
||||||
|
|
||||||
elog(ERROR, "%s: database %s is not owned by you.", command, dbname);
|
elog(ERROR, "%s: database '%s' is not owned by you", command, dbname);
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -317,7 +317,7 @@ stop_vacuum(char *dbpath, char *dbname)
|
|||||||
FreeFile(fp);
|
FreeFile(fp);
|
||||||
if (kill(pid, SIGKILLDAEMON1) < 0)
|
if (kill(pid, SIGKILLDAEMON1) < 0)
|
||||||
{
|
{
|
||||||
elog(ERROR, "can't kill vacuum daemon (pid %d) on %s",
|
elog(ERROR, "can't kill vacuum daemon (pid %d) on '%s'",
|
||||||
pid, dbname);
|
pid, dbname);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user