Custom Query (431 matches)

Filters
 
Or
 
  
 
Columns

Show under each result:


Results (58 - 60 of 431)

Ticket Resolution Summary Owner Reporter
#215 fixed IZ1383: replace_params() function needs improvement Dave Love <d.love@…> crei
Description

[Imported from gridengine issuezilla http://gridengine.sunsource.net/issues/show_bug.cgi?id=1383]

        Issue #:      1383             Platform:     All           Reporter: crei (crei)
       Component:     gridengine          OS:        All
     Subcomponent:    cleanup          Version:      current          CC:    None defined
        Status:       NEW              Priority:     P3
      Resolution:                     Issue type:    ENHANCEMENT
                                   Target milestone: ---
      Assigned to:    andreas (andreas)
      QA Contact:     ernst
          URL:
       * Summary:     replace_params() function needs improvement
   Status whiteboard:
      Attachments:

     Issue 1383 blocks:
   Votes for issue 1383:


   Opened: Fri Dec 3 04:41:00 -0700 2004 
------------------------


Fix from CR-2004-12-02-3 is now disabled.

The dst_len parameter is ignored, because fixing
the missing buffer overwrite problem caused
problems in functions calling replace_params().
Every calling function must set the correct buffer
length of dst. This is currently not the case.
Therefore I disabled the buffer overwrite check by
outcommenting the
 *   "&& dp_pos < max_dst_len" part.

A dstring buffer as paramter should fix the problem.

   ------- Additional comments from sgrell Mon Dec 12 02:41:24 -0700 2005 -------
Changed subcomponent.

Stephan
#218 fixed IZ1412: additional pseudo variable $sge_root for pe definition joga
Description

[Imported from gridengine issuezilla http://gridengine.sunsource.net/issues/show_bug.cgi?id=1412]

        Issue #:      1412             Platform:     All           Reporter: joga (joga)
       Component:     gridengine          OS:        All
     Subcomponent:    execution        Version:      6.0              CC:    None defined
        Status:       NEW              Priority:     P3
      Resolution:                     Issue type:    ENHANCEMENT
                                   Target milestone: ---
      Assigned to:    pollinger (pollinger)
      QA Contact:     pollinger
          URL:
       * Summary:     additional pseudo variable $sge_root for pe definition
   Status whiteboard:
      Attachments:

     Issue 1412 blocks:
   Votes for issue 1412:


   Opened: Fri Jan 7 05:02:00 -0700 2005 
------------------------


In the definition of start_proc_args and
stop_proc_arcs of a parallel environment, pseudo
variables like $pe, $processors, $job_owner etc.
can be specified.

This list of pseudo variables should be extended
by $sge_root and maybe $sge_cell.

This would allow the definition of the start/stop
proc script relative to $SGE_ROOT.

Currently the start/stop proc has to be specified
as absolute path, which makes it impossible to
provide template pe definitions.

Other objects might be affected as well, like the
checkpoint environment definition (migration
command etc.), or the queue definition (prolog,
epilog, terminate_method etc.).

   ------- Additional comments from pollinger Fri Dec 9 09:15:06 -0700 2005 -------
changed subcomponent
#224 fixed IZ1443: code cleanup in builtin_starter.c:start_command() Dave Love <d.love@…> uddeborg
Description

[Imported from gridengine issuezilla http://gridengine.sunsource.net/issues/show_bug.cgi?id=1443]

        Issue #:      1443             Platform:     All     Reporter: uddeborg (uddeborg)
       Component:     gridengine          OS:        All
     Subcomponent:    execution        Version:      6.0u3      CC:    None defined
        Status:       NEW              Priority:     P3
      Resolution:                     Issue type:    TASK
                                   Target milestone: ---
      Assigned to:    andreas (andreas)
      QA Contact:     pollinger
          URL:
       * Summary:     code cleanup in builtin_starter.c:start_command()
   Status whiteboard:
      Attachments:
                      Date/filename:                     Description:                                                              Submitted by:
                      Fri Jan 28 07:36:00 -0700 2005: #  Patch to remove the pre_args_ptr completely. (text/plain)                 uddeborg
                      Fri Jan 28 07:38:00 -0700 2005: #2 Patch to use *pre_args_ptr++ as the technique to assign args (text/plain) uddeborg

     Issue 1443 blocks:
   Votes for issue 1443:


   Opened: Fri Jan 28 07:35:00 -0700 2005 
------------------------


While investigating a problem with submitting
commands (issue 1442) I was confused bu tye use of
a pointer pre_args_ptr in the function
start_command in
source/daemons/shepherd/builtin_starter.c.  It is
set to point to the first element of pre_args, but
is then used as an array.  This original array
pre_args could be used instead.  I get the
impression that somebody intended to assign
successive values with the "*ptr++" idiom, but
then changed his mind and used ordinary array
indexing instead, without removing the pointer
variable from the code.

It was a bit confusing when reading the code, so I
would suggest cleaning it up.  It could be changed
in one of two ways.  Either the pre_args_ptr could
be removed, and all indexing could be done from
the original array.  Or the *ptr++ technique could
be used.  I'll attach two possible patches, one
for each case.

   ------- Additional comments from uddeborg Fri Jan 28 07:36:44 -0700 2005 -------
Created an attachment (id=36)
Patch to remove the pre_args_ptr completely.

   ------- Additional comments from uddeborg Fri Jan 28 07:38:31 -0700 2005 -------
Created an attachment (id=37)
Patch to use *pre_args_ptr++ as the technique to assign args

   ------- Additional comments from uddeborg Fri Jan 28 07:40:33 -0700 2005 -------
Oh, please note that these patches also include the bug fix I suggest
in issue 1442.  Sorry for mixing the two things together.

   ------- Additional comments from sgrell Tue Dec 6 08:17:23 -0700 2005 -------
Changed the Subcomponent.

Stephan

   ------- Additional comments from pollinger Tue Mar 7 02:24:51 -0700 2006 -------
Changed "Issue type" to task
Note: See TracQuery for help on using queries.