Copy from Base:System/cronie based on submit request 49010 from user elvigia OBS-URL: https://build.opensuse.org/request/show/49010 OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/cronie?expand=0&rev=1
79 lines
2.1 KiB
Diff
79 lines
2.1 KiB
Diff
Index: cronie-1.4.3/src/crontab.c
|
|
===================================================================
|
|
--- cronie-1.4.3.orig/src/crontab.c
|
|
+++ cronie-1.4.3/src/crontab.c
|
|
@@ -33,7 +33,7 @@
|
|
# include <selinux/av_permissions.h>
|
|
#endif
|
|
|
|
-#define NHEADER_LINES 0
|
|
+#define NHEADER_LINES 3
|
|
|
|
enum opt_t {opt_unknown, opt_list, opt_delete, opt_edit, opt_replace};
|
|
|
|
@@ -347,7 +347,7 @@ static char *tmp_path() {
|
|
static void edit_cmd(void) {
|
|
char n[MAX_FNAME], q[MAX_TEMPSTR], *editor;
|
|
FILE *f;
|
|
- int ch = '\0', t;
|
|
+ int ch = '\0', t, x;
|
|
struct stat statbuf;
|
|
struct utimbuf utimebuf;
|
|
WAIT_T waiter;
|
|
@@ -399,13 +399,25 @@ static void edit_cmd(void) {
|
|
}
|
|
|
|
Set_LineNum(1)
|
|
- /*
|
|
- * NHEADER_LINES processing removed for clarity
|
|
- * (NHEADER_LINES == 0 in all Red Hat crontabs)
|
|
- */
|
|
- /* copy the rest of the crontab (if any) to the temp file.
|
|
- */
|
|
- if (EOF != ch)
|
|
+
|
|
+ /* ignore the top few comments since we probably put them there.
|
|
+ */
|
|
+ x = 0;
|
|
+ while (EOF != (ch = get_char(f))) {
|
|
+ if ('#' != ch) {
|
|
+ putc(ch, NewCrontab);
|
|
+ break;
|
|
+ }
|
|
+ while (EOF != (ch = get_char(f)))
|
|
+ if (ch == '\n')
|
|
+ break;
|
|
+ if (++x >= NHEADER_LINES)
|
|
+ break;
|
|
+ }
|
|
+
|
|
+ /* copy the rest of the crontab (if any) to the temp file.
|
|
+ */
|
|
+ if (EOF != ch)
|
|
while (EOF != (ch = get_char(f)))
|
|
putc(ch, NewCrontab);
|
|
|
|
@@ -605,6 +617,7 @@ static int replace_cmd(void) {
|
|
int error = 0;
|
|
entry *e;
|
|
uid_t file_owner;
|
|
+ time_t now = time(NULL);
|
|
char **envp = env_init();
|
|
|
|
if (envp == NULL) {
|
|
@@ -636,10 +649,10 @@ static int replace_cmd(void) {
|
|
*
|
|
* VERY IMPORTANT: make sure NHEADER_LINES agrees with this code.
|
|
*/
|
|
- /*fprintf(tmp, "# DO NOT EDIT THIS FILE - edit the master and reinstall.\n");
|
|
- *fprintf(tmp, "# (%s installed on %-24.24s)\n", Filename, ctime(&now));
|
|
- *fprintf(tmp, "# (Cron version %s)\n", CRON_VERSION);
|
|
- */
|
|
+ fprintf(tmp, "# DO NOT EDIT THIS FILE - edit the master and reinstall.\n");
|
|
+ fprintf(tmp, "# (%s installed on %-24.24s)\n", Filename, ctime(&now));
|
|
+ fprintf(tmp, "# (Cronie version %s)\n", CRON_VERSION);
|
|
+
|
|
#ifdef WITH_SELINUX
|
|
if (selinux_context)
|
|
fprintf(tmp, "SELINUX_ROLE_TYPE=%s\n", selinux_context);
|