"
""
""
diff --git a/swad_photo.c b/swad_photo.c
index adb0e310..2aa82caa 100644
--- a/swad_photo.c
+++ b/swad_photo.c
@@ -501,8 +501,9 @@ void Pho_ReqRemoveUsrPhoto (void)
{
/***** Show question and button to remove user's photo *****/
/* Start alert */
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_photo_of_X,
- Gbl.Usrs.Other.UsrDat.FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_photo_of_X,
+ Gbl.Usrs.Other.UsrDat.FullName);
Ale_ShowAlertAndButton1 (Ale_QUESTION,Gbl.Alert.Txt);
/* Show current photo */
@@ -644,7 +645,9 @@ static bool Pho_ReceivePhotoAndDetectFaces (bool ItsMe,const struct UsrData *Usr
WrongType = true;
if (WrongType)
{
- sprintf (Gbl.Alert.Txt,Txt_The_file_is_not_X,"jpg");
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_file_is_not_X,
+ "jpg");
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
return false;
}
@@ -726,9 +729,10 @@ static bool Pho_ReceivePhotoAndDetectFaces (bool ItsMe,const struct UsrData *Usr
NumFacesTotal = NumFacesGreen = NumFacesRed = 0;
break;
default: // Error
- sprintf (Gbl.Alert.Txt,"Photo could not be processed successfully.
"
- "Error code returned by the program of processing: %d",
- ReturnCode);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Photo could not be processed successfully.
"
+ "Error code returned by the program of processing: %d",
+ ReturnCode);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
break;
}
@@ -760,24 +764,28 @@ static bool Pho_ReceivePhotoAndDetectFaces (bool ItsMe,const struct UsrData *Usr
if (NumFacesRed == 0)
{
Gbl.Alert.Type = Ale_SUCCESS;
- sprintf (Gbl.Alert.Txt,Txt_X_faces_marked_in_green_have_been_detected_,
- NumFacesGreen);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_X_faces_marked_in_green_have_been_detected_,
+ NumFacesGreen);
}
else if (NumFacesGreen == 0)
{
Gbl.Alert.Type = Ale_WARNING;
- sprintf (Gbl.Alert.Txt,Txt_X_faces_marked_in_red_have_been_detected_,
- NumFacesRed);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_X_faces_marked_in_red_have_been_detected_,
+ NumFacesRed);
}
else // NumFacesGreen > 0
{
Gbl.Alert.Type = Ale_SUCCESS;
if (NumFacesGreen == 1)
- sprintf (Gbl.Alert.Txt,Txt_X_faces_have_been_detected_in_front_position_1_Z_,
- NumFacesTotal,NumFacesRed);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_X_faces_have_been_detected_in_front_position_1_Z_,
+ NumFacesTotal,NumFacesRed);
else
- sprintf (Gbl.Alert.Txt,Txt_X_faces_have_been_detected_in_front_position_Y_Z_,
- NumFacesTotal,NumFacesGreen,NumFacesRed);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_X_faces_have_been_detected_in_front_position_Y_Z_,
+ NumFacesTotal,NumFacesGreen,NumFacesRed);
}
}
diff --git a/swad_place.c b/swad_place.c
index e0685b9f..2905b57c 100644
--- a/swad_place.c
+++ b/swad_place.c
@@ -577,8 +577,9 @@ void Plc_RemovePlace (void)
DB_QueryDELETE (Query,"can not remove a place");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Place_X_removed,
- Plc.FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Place_X_removed,
+ Plc.FullName);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -652,8 +653,9 @@ static void Plc_RenamePlace (Cns_ShrtOrFullName_t ShrtOrFullName)
/***** Check if new name is empty *****/
if (!NewPlcName[0])
{
- sprintf (Gbl.Alert.Txt,Txt_You_can_not_leave_the_name_of_the_place_X_empty,
- CurrentPlcName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_You_can_not_leave_the_name_of_the_place_X_empty,
+ CurrentPlcName);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else
@@ -664,8 +666,9 @@ static void Plc_RenamePlace (Cns_ShrtOrFullName_t ShrtOrFullName)
/***** If place was in database... *****/
if (Plc_CheckIfPlaceNameExists (ParamName,NewPlcName,Plc->PlcCod))
{
- sprintf (Gbl.Alert.Txt,Txt_The_place_X_already_exists,
- NewPlcName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_place_X_already_exists,
+ NewPlcName);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else
@@ -674,15 +677,17 @@ static void Plc_RenamePlace (Cns_ShrtOrFullName_t ShrtOrFullName)
Plc_UpdatePlcNameDB (Plc->PlcCod,FieldName,NewPlcName);
/* Write message to show the change made */
- sprintf (Gbl.Alert.Txt,Txt_The_place_X_has_been_renamed_as_Y,
- CurrentPlcName,NewPlcName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_place_X_has_been_renamed_as_Y,
+ CurrentPlcName,NewPlcName);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
}
else // The same name
{
- sprintf (Gbl.Alert.Txt,Txt_The_name_of_the_place_X_has_not_changed,
- CurrentPlcName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_name_of_the_place_X_has_not_changed,
+ CurrentPlcName);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
}
@@ -839,14 +844,16 @@ void Plc_RecFormNewPlace (void)
/***** If name of place was in database... *****/
if (Plc_CheckIfPlaceNameExists ("ShortName",Plc->ShrtName,-1L))
{
- sprintf (Gbl.Alert.Txt,Txt_The_place_X_already_exists,
- Plc->ShrtName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_place_X_already_exists,
+ Plc->ShrtName);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else if (Plc_CheckIfPlaceNameExists ("FullName",Plc->FullName,-1L))
{
- sprintf (Gbl.Alert.Txt,Txt_The_place_X_already_exists,
- Plc->FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_place_X_already_exists,
+ Plc->FullName);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else // Add new place to database
@@ -879,7 +886,8 @@ static void Plc_CreatePlace (struct Place *Plc)
DB_QueryINSERT (Query,"can not create place");
/***** Write success message *****/
- sprintf (Gbl.Alert.Txt,Txt_Created_new_place_X,
- Plc->FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Created_new_place_X,
+ Plc->FullName);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
diff --git a/swad_plugin.c b/swad_plugin.c
index ab4411e9..af644840 100644
--- a/swad_plugin.c
+++ b/swad_plugin.c
@@ -501,8 +501,9 @@ void Plg_RemovePlugin (void)
DB_QueryDELETE (Query,"can not remove a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Plugin_X_removed,
- Plg.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Plugin_X_removed,
+ Plg.Name);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show the form again *****/
@@ -539,8 +540,9 @@ void Plg_RenamePlugin (void)
/***** Check if new name is empty *****/
if (!NewPlgName[0])
{
- sprintf (Gbl.Alert.Txt,Txt_You_can_not_leave_the_name_of_the_plugin_X_empty,
- Plg->Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_You_can_not_leave_the_name_of_the_plugin_X_empty,
+ Plg->Name);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else
@@ -551,8 +553,9 @@ void Plg_RenamePlugin (void)
/***** If plugin was in database... *****/
if (Plg_CheckIfPluginNameExists (NewPlgName,Plg->PlgCod))
{
- sprintf (Gbl.Alert.Txt,Txt_The_plugin_X_already_exists,
- NewPlgName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_plugin_X_already_exists,
+ NewPlgName);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else
@@ -563,15 +566,17 @@ void Plg_RenamePlugin (void)
DB_QueryUPDATE (Query,"can not update the name of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_plugin_X_has_been_renamed_as_Y,
- Plg->Name,NewPlgName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_plugin_X_has_been_renamed_as_Y,
+ Plg->Name,NewPlgName);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
}
else // The same name
{
- sprintf (Gbl.Alert.Txt,Txt_The_name_of_the_plugin_X_has_not_changed,
- Plg->Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_name_of_the_plugin_X_has_not_changed,
+ Plg->Name);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
}
@@ -628,8 +633,9 @@ void Plg_ChangePlgDescription (void)
DB_QueryUPDATE (Query,"can not update the description of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_new_description_is_X,
- NewDescription);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_new_description_is_X,
+ NewDescription);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -672,8 +678,9 @@ void Plg_ChangePlgLogo (void)
DB_QueryUPDATE (Query,"can not update the logo of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_new_logo_is_X,
- NewLogo);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_new_logo_is_X,
+ NewLogo);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -716,8 +723,9 @@ void Plg_ChangePlgAppKey (void)
DB_QueryUPDATE (Query,"can not update the application key of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_new_logo_is_X, // TODO!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
- NewAppKey);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_new_logo_is_X, // TODO!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+ NewAppKey);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -760,8 +768,9 @@ void Plg_ChangePlgURL (void)
DB_QueryUPDATE (Query,"can not update the URL of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_new_URL_is_X,
- NewURL);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_new_URL_is_X,
+ NewURL);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -804,8 +813,9 @@ void Plg_ChangePlgIP (void)
DB_QueryUPDATE (Query,"can not update the IP address of a plugin");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_new_IP_address_is_X,
- NewIP);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_new_IP_address_is_X,
+ NewIP);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -1013,8 +1023,9 @@ void Plg_RecFormNewPlg (void)
/***** If name of plugin was in database... *****/
if (Plg_CheckIfPluginNameExists (Plg->Name,-1L))
{
- sprintf (Gbl.Alert.Txt,Txt_The_plugin_X_already_exists,
- Plg->Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_plugin_X_already_exists,
+ Plg->Name);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else // Add new plugin to database
@@ -1053,8 +1064,9 @@ static void Plg_CreatePlugin (struct Plugin *Plg)
DB_QueryINSERT (Query,"can not create plugin");
/***** Write success message *****/
- sprintf (Gbl.Alert.Txt,Txt_Created_new_plugin_X,
- Plg->Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Created_new_plugin_X,
+ Plg->Name);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
diff --git a/swad_project.c b/swad_project.c
index 1a36b301..266759ea 100644
--- a/swad_project.c
+++ b/swad_project.c
@@ -1981,9 +1981,10 @@ static void Prj_AddUsrToProject (Prj_RoleInProject_t RoleInProject)
Prj_FlushCacheMyRoleInProject ();
/* Show success alert */
- sprintf (Gbl.Alert.Txt,Txt_THE_USER_X_has_been_enroled_as_a_Y_in_the_project,
- Gbl.Usrs.Other.UsrDat.FullName,
- Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject]);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_THE_USER_X_has_been_enroled_as_a_Y_in_the_project,
+ Gbl.Usrs.Other.UsrDat.FullName,
+ Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject]);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -2054,11 +2055,11 @@ static void Prj_ReqRemUsrFromPrj (Prj_RoleInProject_t RoleInProject)
/***** Show question and button to remove user as administrator *****/
/* Start alert */
- sprintf (Gbl.Alert.Txt,
- ItsMe ? Txt_Do_you_really_want_to_be_removed_as_a_X_from_the_project_Y :
- Txt_Do_you_really_want_to_remove_the_following_user_as_a_X_from_the_project_Y,
- Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject],
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ ItsMe ? Txt_Do_you_really_want_to_be_removed_as_a_X_from_the_project_Y :
+ Txt_Do_you_really_want_to_remove_the_following_user_as_a_X_from_the_project_Y,
+ Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject],
+ Prj.Title);
Ale_ShowAlertAndButton1 (Ale_QUESTION,Gbl.Alert.Txt);
/* Show user's record */
@@ -2146,10 +2147,11 @@ static void Prj_RemUsrFromPrj (Prj_RoleInProject_t RoleInProject)
Prj_FlushCacheMyRoleInProject ();
/***** Show success alert *****/
- sprintf (Gbl.Alert.Txt,Txt_THE_USER_X_has_been_removed_as_a_Y_from_the_project_Z,
- Gbl.Usrs.Other.UsrDat.FullName,
- Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject],
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_THE_USER_X_has_been_removed_as_a_Y_from_the_project_Z,
+ Gbl.Usrs.Other.UsrDat.FullName,
+ Txt_PROJECT_ROLES_SINGUL_abc[RoleInProject],
+ Prj.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -2731,8 +2733,9 @@ void Prj_ReqRemProject (void)
{
/***** Show question and button to remove the project *****/
Gbl.Prjs.PrjCod = Prj.PrjCod;
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_project_X,
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_project_X,
+ Prj.Title);
Ale_ShowAlertAndButton (Ale_QUESTION,Gbl.Alert.Txt,
ActRemPrj,NULL,NULL,Prj_PutCurrentParams,
Btn_REMOVE_BUTTON,Txt_Remove_project);
@@ -2797,8 +2800,9 @@ void Prj_RemoveProject (void)
Fil_RemoveTree (PathRelPrj);
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Project_X_removed,
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Project_X_removed,
+ Prj.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -2841,8 +2845,9 @@ void Prj_HideProject (void)
DB_QueryUPDATE (Query,"can not hide project");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Project_X_is_now_hidden,
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Project_X_is_now_hidden,
+ Prj.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -2885,8 +2890,9 @@ void Prj_ShowProject (void)
DB_QueryUPDATE (Query,"can not show project");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Project_X_is_now_visible,
- Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Project_X_is_now_visible,
+ Prj.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else
@@ -3305,7 +3311,9 @@ void Prj_RecFormProject (void)
Prj_CreateProject (&Prj); // Add new project to database
/* Write success message */
- sprintf (Gbl.Alert.Txt,Txt_Created_new_project_X,Prj.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Created_new_project_X,
+ Prj.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
else if (NewProjectIsCorrect)
diff --git a/swad_record.c b/swad_record.c
index 80804166..e6c1b594 100644
--- a/swad_record.c
+++ b/swad_record.c
@@ -202,8 +202,9 @@ void Rec_ReqEditRecordFields (void)
}
else // No fields of records found for current course in the database
{
- sprintf (Gbl.Alert.Txt,Txt_There_are_no_record_fields_in_the_course_X,
- Gbl.CurrentCrs.Crs.FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_are_no_record_fields_in_the_course_X,
+ Gbl.CurrentCrs.Crs.FullName);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
@@ -482,8 +483,9 @@ void Rec_ReceiveFormField (void)
/***** If the field already was in the database... *****/
if (Rec_CheckIfRecordFieldIsRepeated (Gbl.CurrentCrs.Records.Field.Name))
{
- sprintf (Gbl.Alert.Txt,Txt_The_record_field_X_already_exists,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_record_field_X_already_exists,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_ERROR,Gbl.Alert.Txt);
}
else // Add the new field to the database
@@ -586,8 +588,9 @@ void Rec_CreateRecordField (void)
DB_QueryINSERT (Query,"can not create field of record");
/***** Write message of success *****/
- sprintf (Gbl.Alert.Txt,Txt_Created_new_record_field_X,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Created_new_record_field_X,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -655,8 +658,9 @@ void Rec_AskConfirmRemFieldWithRecords (unsigned NumRecords)
&Gbl.CurrentCrs.Records.Field.Visibility);
/***** Show question and button to remove my photo *****/
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_field_X_from_the_records_of_X,
- Gbl.CurrentCrs.Records.Field.Name,Gbl.CurrentCrs.Crs.FullName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_field_X_from_the_records_of_X,
+ Gbl.CurrentCrs.Records.Field.Name,Gbl.CurrentCrs.Crs.FullName);
if (NumRecords == 1)
Str_Concat (Gbl.Alert.Txt,Txt_this_field_is_filled_in_the_record_of_one_student,
Ale_MAX_BYTES_ALERT);
@@ -701,8 +705,9 @@ void Rec_RemoveFieldFromDB (void)
DB_QueryDELETE (Query,"can not remove field of record");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Record_field_X_removed,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Record_field_X_removed,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show the form again *****/
@@ -807,8 +812,9 @@ void Rec_RenameField (void)
/***** Check if new name is empty *****/
if (!NewFieldName[0])
{
- sprintf (Gbl.Alert.Txt,Txt_You_can_not_leave_the_name_of_the_field_X_empty,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_You_can_not_leave_the_name_of_the_field_X_empty,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_ERROR,Gbl.Alert.Txt);
}
else
@@ -820,8 +826,9 @@ void Rec_RenameField (void)
/***** If the group ya estaba in the database... *****/
if (Rec_CheckIfRecordFieldIsRepeated (NewFieldName))
{
- sprintf (Gbl.Alert.Txt,Txt_The_record_field_X_already_exists,
- NewFieldName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_record_field_X_already_exists,
+ NewFieldName);
Ale_ShowAlert (Ale_ERROR,Gbl.Alert.Txt);
}
else
@@ -833,15 +840,17 @@ void Rec_RenameField (void)
DB_QueryUPDATE (Query,"can not update name of field of record");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_record_field_X_has_been_renamed_as_Y,
- Gbl.CurrentCrs.Records.Field.Name,NewFieldName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_record_field_X_has_been_renamed_as_Y,
+ Gbl.CurrentCrs.Records.Field.Name,NewFieldName);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
}
else // The same name
{
- sprintf (Gbl.Alert.Txt,Txt_The_name_of_the_field_X_has_not_changed,
- NewFieldName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_name_of_the_field_X_has_not_changed,
+ NewFieldName);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
}
@@ -882,8 +891,9 @@ void Rec_ChangeLinesField (void)
(this happens when return is pressed without changes) *****/
if (Gbl.CurrentCrs.Records.Field.NumLines == NewNumLines)
{
- sprintf (Gbl.Alert.Txt,Txt_The_number_of_editing_lines_in_the_record_field_X_has_not_changed,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_number_of_editing_lines_in_the_record_field_X_has_not_changed,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
else
@@ -895,8 +905,9 @@ void Rec_ChangeLinesField (void)
DB_QueryUPDATE (Query,"can not update the number of lines of a field of record");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_From_now_on_the_number_of_editing_lines_of_the_field_X_is_Y,
- Gbl.CurrentCrs.Records.Field.Name,NewNumLines);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_From_now_on_the_number_of_editing_lines_of_the_field_X_is_Y,
+ Gbl.CurrentCrs.Records.Field.Name,NewNumLines);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -935,8 +946,9 @@ void Rec_ChangeVisibilityField (void)
(this happens whe return is pressed without changes in the form) *****/
if (Gbl.CurrentCrs.Records.Field.Visibility == NewVisibility)
{
- sprintf (Gbl.Alert.Txt,Txt_The_visibility_of_the_record_field_X_has_not_changed,
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_visibility_of_the_record_field_X_has_not_changed,
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
else
@@ -948,8 +960,9 @@ void Rec_ChangeVisibilityField (void)
DB_QueryUPDATE (Query,"can not update the visibility of a field of record");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_RECORD_FIELD_VISIBILITY_MSG[NewVisibility],
- Gbl.CurrentCrs.Records.Field.Name);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_RECORD_FIELD_VISIBILITY_MSG[NewVisibility],
+ Gbl.CurrentCrs.Records.Field.Name);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
diff --git a/swad_session.c b/swad_session.c
index 7fbefc8e..aee201eb 100644
--- a/swad_session.c
+++ b/swad_session.c
@@ -485,9 +485,10 @@ unsigned Ses_GetHiddenParFromDB (Act_Action_t NextAction,
if (ParameterIsTooBig)
{
- sprintf (Gbl.Alert.Txt,"Hidden parameter %s too large,"
- " it exceed the maximum allowed size (%lu bytes).",
- ParamName,(unsigned long) MaxBytes);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Hidden parameter %s too large,"
+ " it exceed the maximum allowed size (%lu bytes).",
+ ParamName,(unsigned long) MaxBytes);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
diff --git a/swad_statistic.c b/swad_statistic.c
index bf16c619..1b241421 100644
--- a/swad_statistic.c
+++ b/swad_statistic.c
@@ -1041,8 +1041,9 @@ static void Sta_ShowHits (Sta_GlobalOrCourseAccesses_t GlobalOrCourse)
Gbl.Usrs.Me.Role.Logged == Rol_SYS_ADM;
if (!ICanQueryWholeRange && NumDays > Cfg_DAYS_IN_RECENT_LOG)
{
- sprintf (Gbl.Alert.Txt,Txt_The_date_range_must_be_less_than_or_equal_to_X_days,
- Cfg_DAYS_IN_RECENT_LOG);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_date_range_must_be_less_than_or_equal_to_X_days,
+ Cfg_DAYS_IN_RECENT_LOG);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt); // ...write warning message and show the form again
return;
}
diff --git a/swad_string.c b/swad_string.c
index c84bc467..1039138e 100644
--- a/swad_string.c
+++ b/swad_string.c
@@ -1038,7 +1038,9 @@ void Str_ChangeFormat (Str_ChangeFrom_t ChangeFrom,Str_ChangeTo_t ChangeTo,
/*
if (Gbl.Usrs.Me.Roles.LoggedRole == Rol_SYS_ADM)
{
- sprintf (Gbl.Alert.Txt,"Str_ChangeFormat ("%s")",Str);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Str_ChangeFormat ("%s")",
+ Str);
Lay_ShowAlert (Lay_INFO,Gbl.Alert.Txt);
}
*/
@@ -2880,8 +2882,9 @@ void Str_Copy (char *Dst,const char *Src,size_t DstSize)
/***** Check if buffer has enough space for source *****/
if (LengthSrc > DstSize)
{
- sprintf (Gbl.Alert.Txt,"Trying to copy %lu chars into a %lu-chars buffer.",
- LengthSrc,DstSize);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Trying to copy %lu chars into a %lu-chars buffer.",
+ LengthSrc,DstSize);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
@@ -2903,8 +2906,9 @@ void Str_Concat (char *Dst,const char *Src,size_t DstSize)
LengthDst = strlen (Dst);
if (LengthDst > DstSize)
{
- sprintf (Gbl.Alert.Txt,"%lu-chars buffer has %lu chars!",
- DstSize,LengthDst);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "%lu-chars buffer has %lu chars!",
+ DstSize,LengthDst);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
@@ -2914,8 +2918,9 @@ void Str_Concat (char *Dst,const char *Src,size_t DstSize)
LengthSrc = strlen (Src);
if (FreeSpace < LengthSrc)
{
- sprintf (Gbl.Alert.Txt,"Trying to concatenate %lu chars to a %lu-chars buffer with free space for only %lu chars!",
- LengthSrc,DstSize,FreeSpace);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Trying to concatenate %lu chars to a %lu-chars buffer with free space for only %lu chars!",
+ LengthSrc,DstSize,FreeSpace);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
diff --git a/swad_survey.c b/swad_survey.c
index 625b28c5..84f43d97 100644
--- a/swad_survey.c
+++ b/swad_survey.c
@@ -1508,8 +1508,9 @@ void Svy_AskRemSurvey (void)
/***** Show question and button to remove survey *****/
Gbl.Svys.SvyCodToEdit = Svy.SvyCod;
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_survey_X,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_survey_X,
+ Svy.Title);
Ale_ShowAlertAndButton (Ale_QUESTION,Gbl.Alert.Txt,
ActRemSvy,NULL,NULL,Svy_PutParams,
Btn_REMOVE_BUTTON,Txt_Remove_survey);
@@ -1571,8 +1572,9 @@ void Svy_RemoveSurvey (void)
Ntf_MarkNotifAsRemoved (Ntf_EVENT_SURVEY,Svy.SvyCod);
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Survey_X_removed,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Survey_X_removed,
+ Svy.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show surveys again *****/
@@ -1607,8 +1609,9 @@ void Svy_AskResetSurvey (void)
Lay_ShowErrorAndExit ("You can not reset this survey.");
/***** Ask for confirmation of reset *****/
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_reset_the_survey_X,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_reset_the_survey_X,
+ Svy.Title);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
/***** Button of confirmation of reset *****/
@@ -1669,8 +1672,9 @@ void Svy_ResetSurvey (void)
DB_QueryUPDATE (Query,"can not reset answers of a survey");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Survey_X_reset,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Survey_X_reset,
+ Svy.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show surveys again *****/
@@ -1706,8 +1710,9 @@ void Svy_HideSurvey (void)
DB_QueryUPDATE (Query,"can not hide survey");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Survey_X_is_now_hidden,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Survey_X_is_now_hidden,
+ Svy.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show surveys again *****/
@@ -1743,8 +1748,9 @@ void Svy_UnhideSurvey (void)
DB_QueryUPDATE (Query,"can not show survey");
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_Survey_X_is_now_visible,
- Svy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Survey_X_is_now_visible,
+ Svy.Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Show surveys again *****/
@@ -2195,8 +2201,9 @@ void Svy_RecFormSurvey (void)
if (Svy_CheckIfSimilarSurveyExists (&NewSvy))
{
NewSurveyIsCorrect = false;
- sprintf (Gbl.Alert.Txt,Txt_Already_existed_a_survey_with_the_title_X,
- NewSvy.Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Already_existed_a_survey_with_the_title_X,
+ NewSvy.Title);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
}
@@ -2281,8 +2288,9 @@ static void Svy_CreateSurvey (struct Survey *Svy,const char *Txt)
Svy_CreateGrps (Svy->SvyCod);
/***** Write success message *****/
- sprintf (Gbl.Alert.Txt,Txt_Created_new_survey_X,
- Svy->Title);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Created_new_survey_X,
+ Svy->Title);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -2915,7 +2923,8 @@ static bool Svy_AllocateTextChoiceAnswer (struct SurveyQuestion *SvyQst,
Svy_FreeTextChoiceAnswer (SvyQst,NumAns);
if ((SvyQst->AnsChoice[NumAns].Text = (char *) malloc (Svy_MAX_BYTES_ANSWER + 1)) == NULL)
{
- sprintf (Gbl.Alert.Txt,"Not enough memory to store answer.");
+ Str_Copy (Gbl.Alert.Txt,"Not enough memory to store answer.",
+ Ale_MAX_BYTES_ALERT);
return false;
}
SvyQst->AnsChoice[NumAns].Text[0] = '\0';
@@ -3586,8 +3595,9 @@ void Svy_RequestRemoveQst (void)
/***** Show question and button to remove question *****/
Gbl.Svys.SvyCodToEdit = SvyCod;
Gbl.Svys.SvyQstCodToEdit = SvyQst.QstCod;
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_question_X,
- (unsigned long) (SvyQst.QstInd + 1));
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_question_X,
+ (unsigned long) (SvyQst.QstInd + 1));
Ale_ShowAlertAndButton (Ale_QUESTION,Gbl.Alert.Txt,
ActRemSvyQst,NULL,NULL,Svy_PutParamsRemoveOneQst,
Btn_REMOVE_BUTTON,Txt_Remove_question);
diff --git a/swad_test.c b/swad_test.c
index 00a486d2..4b8648e6 100644
--- a/swad_test.c
+++ b/swad_test.c
@@ -584,13 +584,15 @@ void Tst_AssessTest (void)
Tst_SetTstStatus (NumTst,Tst_STATUS_ASSESSED);
break;
case Tst_STATUS_ASSESSED:
- sprintf (Gbl.Alert.Txt,Txt_The_test_X_has_already_been_assessed_previously,
- NumTst);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_test_X_has_already_been_assessed_previously,
+ NumTst);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
break;
case Tst_STATUS_ERROR:
- sprintf (Gbl.Alert.Txt,Txt_There_was_an_error_in_assessing_the_test_X,
- NumTst);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_was_an_error_in_assessing_the_test_X,
+ NumTst);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
break;
}
@@ -698,14 +700,15 @@ static bool Tst_CheckIfNextTstAllowed (void)
if (NumSecondsFromNowToNextAccTst > 0)
{
/***** Write warning *****/
- sprintf (Gbl.Alert.Txt,"%s:
."
- "",
- Txt_You_can_not_take_a_new_test_until,
- (long) TimeNextTestUTC,
- (unsigned) Gbl.Prefs.DateFormat,Txt_Today);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "%s:
."
+ "",
+ Txt_You_can_not_take_a_new_test_until,
+ (long) TimeNextTestUTC,
+ (unsigned) Gbl.Prefs.DateFormat,Txt_Today);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
return false;
@@ -1552,8 +1555,9 @@ void Tst_RenameTag (void)
/***** Check that the new tag is not empty *****/
if (!NewTagTxt[0]) // New tag empty
{
- sprintf (Gbl.Alert.Txt,Txt_You_can_not_leave_the_name_of_the_tag_X_empty,
- OldTagTxt);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_You_can_not_leave_the_name_of_the_tag_X_empty,
+ OldTagTxt);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else // New tag not empty
@@ -1564,8 +1568,9 @@ void Tst_RenameTag (void)
// This happens when user press INTRO
// without changing anything in the form.
{
- sprintf (Gbl.Alert.Txt,Txt_The_tag_X_has_not_changed,
- NewTagTxt);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_tag_X_has_not_changed,
+ NewTagTxt);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
else // The old and the new tag
@@ -1648,8 +1653,9 @@ void Tst_RenameTag (void)
}
/***** Write message to show the change made *****/
- sprintf (Gbl.Alert.Txt,Txt_The_tag_X_has_been_renamed_as_Y,
- OldTagTxt,NewTagTxt);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_tag_X_has_been_renamed_as_Y,
+ OldTagTxt,NewTagTxt);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
}
@@ -4760,8 +4766,9 @@ static bool Tst_GetParamsTst (Tst_ActionToDoWithQuestions_t ActionToDoWithQuesti
if (Gbl.Test.NumQsts < Gbl.Test.Config.Min ||
Gbl.Test.NumQsts > Gbl.Test.Config.Max)
{
- sprintf (Gbl.Alert.Txt,Txt_The_number_of_questions_must_be_in_the_interval_X,
- Gbl.Test.Config.Min,Gbl.Test.Config.Max);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_number_of_questions_must_be_in_the_interval_X,
+ Gbl.Test.Config.Min,Gbl.Test.Config.Max);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
Error = true;
}
@@ -5419,13 +5426,15 @@ int Tst_AllocateTextChoiceAnswer (unsigned NumOpt)
if ((Gbl.Test.Answer.Options[NumOpt].Text =
(char *) malloc (Tst_MAX_BYTES_ANSWER_OR_FEEDBACK + 1)) == NULL)
{
- sprintf (Gbl.Alert.Txt,"Not enough memory to store answer.");
+ Str_Copy (Gbl.Alert.Txt,"Not enough memory to store answer.",
+ Ale_MAX_BYTES_ALERT);
return 0;
}
if ((Gbl.Test.Answer.Options[NumOpt].Feedback =
(char *) malloc (Tst_MAX_BYTES_ANSWER_OR_FEEDBACK + 1)) == NULL)
{
- sprintf (Gbl.Alert.Txt,"Not enough memory to store feedback.");
+ Str_Copy (Gbl.Alert.Txt,"Not enough memory to store feedback.",
+ Ale_MAX_BYTES_ALERT);
return 0;
}
@@ -6325,8 +6334,9 @@ void Tst_RequestRemoveQst (void)
Lay_ShowErrorAndExit ("Wrong test parameters.");
/***** Show question and button to remove question *****/
- sprintf (Gbl.Alert.Txt,Txt_Do_you_really_want_to_remove_the_question_X,
- (unsigned long) Gbl.Test.QstCod);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Do_you_really_want_to_remove_the_question_X,
+ (unsigned long) Gbl.Test.QstCod);
if (EditingOnlyThisQst)
Ale_ShowAlertAndButton (Ale_QUESTION,Gbl.Alert.Txt,
ActRemTstQst,NULL,NULL,
@@ -6448,10 +6458,10 @@ void Tst_ChangeShuffleQst (void)
DB_QueryUPDATE (Query,"can not update the shuffle type of a question");
/***** Write message *****/
- sprintf (Gbl.Alert.Txt,
- Shuffle ? Txt_The_answers_of_the_question_with_code_X_will_appear_shuffled :
- Txt_The_answers_of_the_question_with_code_X_will_appear_without_shuffling,
- Gbl.Test.QstCod);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Shuffle ? Txt_The_answers_of_the_question_with_code_X_will_appear_shuffled :
+ Txt_The_answers_of_the_question_with_code_X_will_appear_without_shuffling,
+ Gbl.Test.QstCod);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
/***** Continue editing questions *****/
diff --git a/swad_test_import.c b/swad_test_import.c
index d6fa38d4..fd607945 100644
--- a/swad_test_import.c
+++ b/swad_test_import.c
@@ -426,7 +426,9 @@ void TsI_ImportQstsFromXML (void)
if (WrongType)
{
- sprintf (Gbl.Alert.Txt,Txt_The_file_is_not_X,"xml");
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_file_is_not_X,
+ "xml");
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
else
diff --git a/swad_user.c b/swad_user.c
index fd662495..5f413605 100644
--- a/swad_user.c
+++ b/swad_user.c
@@ -2473,12 +2473,14 @@ void Usr_WelcomeUsr (void)
Gbl.Prefs.PathIconSet,Cfg_ICON_128x128,
Txt_Happy_birthday,
Txt_Happy_birthday);
- sprintf (Gbl.Alert.Txt,Txt_Welcome_X_and_happy_birthday[Gbl.Usrs.Me.UsrDat.Sex],
- Gbl.Usrs.Me.UsrDat.FirstName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Welcome_X_and_happy_birthday[Gbl.Usrs.Me.UsrDat.Sex],
+ Gbl.Usrs.Me.UsrDat.FirstName);
}
if (!CongratulateMyBirthday)
- sprintf (Gbl.Alert.Txt,Txt_Welcome_X[Gbl.Usrs.Me.UsrDat.Sex],
- Gbl.Usrs.Me.UsrDat.FirstName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_Welcome_X[Gbl.Usrs.Me.UsrDat.Sex],
+ Gbl.Usrs.Me.UsrDat.FirstName);
Ale_ShowAlert (Ale_INFO,Gbl.Alert.Txt);
}
else
@@ -2749,8 +2751,9 @@ unsigned Usr_GetParamOtherUsrIDNickOrEMailAndGetUsrCods (struct ListUsrCods *Lis
if (Wrong)
{
/***** String is not a valid user's nickname, email or ID *****/
- sprintf (Gbl.Alert.Txt,Txt_The_ID_nickname_or_email_X_is_not_valid,
- Gbl.Usrs.Other.UsrDat.UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_ID_nickname_or_email_X_is_not_valid,
+ Gbl.Usrs.Other.UsrDat.UsrIDNickOrEmail);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
@@ -3182,8 +3185,9 @@ static void Usr_ShowAlertThereAreMoreThanOneUsr (void)
Gbl.Action.Act = ActFrmLogIn;
Tab_SetCurrentTab ();
- sprintf (Gbl.Alert.Txt,Txt_There_are_more_than_one_user_with_the_ID_X_Please_type_a_nick_or_email,
- Gbl.Usrs.Me.UsrIdLogin);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_are_more_than_one_user_with_the_ID_X_Please_type_a_nick_or_email,
+ Gbl.Usrs.Me.UsrIdLogin);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
@@ -3277,9 +3281,10 @@ void Usr_ShowFormsLogoutAndRole (void)
/***** Write message with my new logged role *****/
if (Gbl.Usrs.Me.Role.HasChanged)
{
- sprintf (Gbl.Alert.Txt,Txt_You_are_now_LOGGED_IN_as_X,
- Txt_logged[Gbl.Usrs.Me.UsrDat.Sex],
- Txt_ROLES_SINGUL_abc[Gbl.Usrs.Me.Role.Logged][Gbl.Usrs.Me.UsrDat.Sex]);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_You_are_now_LOGGED_IN_as_X,
+ Txt_logged[Gbl.Usrs.Me.UsrDat.Sex],
+ Txt_ROLES_SINGUL_abc[Gbl.Usrs.Me.Role.Logged][Gbl.Usrs.Me.UsrDat.Sex]);
Ale_ShowAlert (Ale_SUCCESS,Gbl.Alert.Txt);
}
@@ -5130,8 +5135,9 @@ static void Usr_GetListUsrsFromQuery (const char *Query,Rol_Role_t Role,Sco_Scop
{
if (Gbl.Usrs.LstUsrs[Role].NumUsrs > Cfg_MAX_USRS_IN_LIST)
{
- sprintf (Gbl.Alert.Txt,Txt_The_list_of_X_users_is_too_large_to_be_displayed,
- Gbl.Usrs.LstUsrs[Role].NumUsrs);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_list_of_X_users_is_too_large_to_be_displayed,
+ Gbl.Usrs.LstUsrs[Role].NumUsrs);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
Abort = true;
}
@@ -5319,7 +5325,9 @@ static void Usr_AllocateUsrsList (Rol_Role_t Role)
/*
if (Gbl.Usrs.Me.Roles.LoggedRole == Rol_SYS_ADM)
{
- sprintf (Gbl.Alert.Txt,"Memory used by list = %lu",(long) sizeof (struct UsrInList) * NumUsrs);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "Memory used by list = %lu",
+ (long) sizeof (struct UsrInList) * NumUsrs);
Lay_ShowAlert (Lay_INFO,Gbl.Alert.Txt);
}
*/
@@ -5380,8 +5388,9 @@ static void Usr_PutButtonToConfirmIWantToSeeBigList (unsigned NumUsrs,const char
extern const char *Txt_Show_anyway;
/***** Show alert and button to confirm that I want to see the big list *****/
- sprintf (Gbl.Alert.Txt,Txt_The_list_of_X_users_is_too_large_to_be_displayed,
- NumUsrs);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_list_of_X_users_is_too_large_to_be_displayed,
+ NumUsrs);
Ale_ShowAlertAndButton (Ale_WARNING,Gbl.Alert.Txt,
Gbl.Action.Act,Usr_USER_LIST_SECTION_ID,OnSubmit,
Usr_PutParamsConfirmIWantToSeeBigList,
@@ -5525,8 +5534,9 @@ bool Usr_GetListMsgRecipientsWrittenExplicitelyBySender (bool WriteErrorMsgs)
{
if (WriteErrorMsgs)
{
- sprintf (Gbl.Alert.Txt,Txt_There_is_no_user_with_nickname_X,
- UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_is_no_user_with_nickname_X,
+ UsrIDNickOrEmail);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
Error = true;
@@ -5544,8 +5554,9 @@ bool Usr_GetListMsgRecipientsWrittenExplicitelyBySender (bool WriteErrorMsgs)
{
if (WriteErrorMsgs)
{
- sprintf (Gbl.Alert.Txt,Txt_There_is_no_user_with_email_X,
- UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_is_no_user_with_email_X,
+ UsrIDNickOrEmail);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
Error = true;
@@ -5572,8 +5583,9 @@ bool Usr_GetListMsgRecipientsWrittenExplicitelyBySender (bool WriteErrorMsgs)
{// TODO: Consider forbid IDs here
if (WriteErrorMsgs)
{
- sprintf (Gbl.Alert.Txt,Txt_There_are_more_than_one_user_with_the_ID_X_Please_type_a_nick_or_email,
- UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_are_more_than_one_user_with_the_ID_X_Please_type_a_nick_or_email,
+ UsrIDNickOrEmail);
Ale_ShowAlert (Ale_ERROR,Gbl.Alert.Txt);
}
Error = true;
@@ -5583,8 +5595,9 @@ bool Usr_GetListMsgRecipientsWrittenExplicitelyBySender (bool WriteErrorMsgs)
{
if (WriteErrorMsgs)
{
- sprintf (Gbl.Alert.Txt,Txt_There_is_no_user_with_ID_nick_or_email_X,
- UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_There_is_no_user_with_ID_nick_or_email_X,
+ UsrIDNickOrEmail);
Ale_ShowAlert (Ale_ERROR,Gbl.Alert.Txt);
}
Error = true;
@@ -5594,8 +5607,9 @@ bool Usr_GetListMsgRecipientsWrittenExplicitelyBySender (bool WriteErrorMsgs)
{
if (WriteErrorMsgs)
{
- sprintf (Gbl.Alert.Txt,Txt_The_ID_nickname_or_email_X_is_not_valid,
- UsrIDNickOrEmail);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ Txt_The_ID_nickname_or_email_X_is_not_valid,
+ UsrIDNickOrEmail);
Ale_ShowAlert (Ale_WARNING,Gbl.Alert.Txt);
}
Error = true;
diff --git a/swad_xml.c b/swad_xml.c
index b2fc2ca8..e819d6af 100644
--- a/swad_xml.c
+++ b/swad_xml.c
@@ -151,12 +151,16 @@ static void XML_GetElement (struct XMLElement *ParentElem)
EndTagNameLength = strcspn (Gbl.XMLPtr,">");
if (ParentElem->TagNameLength != EndTagNameLength)
{
- sprintf (Gbl.Alert.Txt,"XML syntax error. Expect end tag </%s>.",ParentElem->TagName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "XML syntax error. Expect end tag </%s>.",
+ ParentElem->TagName);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
if (strncmp (ParentElem->TagName,Gbl.XMLPtr,EndTagNameLength)) // XML tags are case sensitive
{
- sprintf (Gbl.Alert.Txt,"XML syntax error. Expect end tag </%s>.",ParentElem->TagName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "XML syntax error. Expect end tag </%s>.",
+ ParentElem->TagName);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
@@ -360,8 +364,9 @@ static void XML_GetAttributes (struct XMLElement *Elem)
}
else
{
- sprintf (Gbl.Alert.Txt,"XML syntax error after attribute "%s" inside element "%s".",
- Attribute->AttributeName,Elem->TagName);
+ snprintf (Gbl.Alert.Txt,Ale_MAX_BYTES_ALERT,
+ "XML syntax error after attribute "%s" inside element "%s".",
+ Attribute->AttributeName,Elem->TagName);
Lay_ShowErrorAndExit (Gbl.Alert.Txt);
}
|