aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhinder <hinder@b32723a9-ab3a-4a60-88e2-2e5d99d7c17a>2013-01-07 09:53:51 +0000
committerhinder <hinder@b32723a9-ab3a-4a60-88e2-2e5d99d7c17a>2013-01-07 09:53:51 +0000
commit3080df7a41c2b5d73166540d2b49f64854f6d8dc (patch)
treec9cef4be1b26c13da9bf1745e33fca878659caf1
parent0f5656757a18256678f4dc593a2b13aa8d26a729 (diff)
Add TODO item relating to checking the return value of CCTK_ParameterFilename
git-svn-id: http://svn.cactuscode.org/arrangements/CactusBase/IOUtil/trunk@294 b32723a9-ab3a-4a60-88e2-2e5d99d7c17a
-rw-r--r--src/Startup.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/src/Startup.c b/src/Startup.c
index 7bb69ca..3acba9f 100644
--- a/src/Startup.c
+++ b/src/Startup.c
@@ -95,6 +95,13 @@ int IOUtil_Startup (void)
/* advertise the parameter file */
parfile[0] = 0;
+
+ /* TODO: check the returned size; the function silently truncates
+ * the filename if it doesn't fit in the buffer; I can't think of a
+ * situation where this is useful. Even better, modify
+ * CCTK_ParameterFilename to allocate memory for the string, like
+ * many other Cactus functions do. */
+
CCTK_ParameterFilename (sizeof (parfile), parfile);
advertised_file.slice = "";
advertised_file.thorn = CCTK_THORNSTRING;