# Review: Marshal bugs affecting rubygems

**URL:** <https://www.ruby-forum.com/t/review-marshal-bugs-affecting-rubygems/163765>\
**Category:** IronRuby\
**Created:** [March 24, 2009, 6:24pm UTC](https://www.ruby-forum.com/t/review-marshal-bugs-affecting-rubygems/163765 "2009-03-24T18:24:29Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![Shri\_B](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/s/898d66/32.png) [@Shri\_B](https://www.ruby-forum.com/u/Shri_B)\
**Post date:** [March 24, 2009, 6:24pm UTC](https://www.ruby-forum.com/t/review-marshal-bugs-affecting-rubygems/163765/1 "2009-03-24T18:24:29Z")

</div>

GitDiff.bat d522059c42607fd2ea1cf8b39fc78006f5e4383d  
79298efe4e30261ff76b3f989fc2d0002a47c9d6, or

> <https://github.com/shri-zz/ironruby/commit/79298efe4e30261ff76b3f989fc2d0002a47c9d6>
>
> TimeOps.load had a bug causing local time to be loaded as utc time.
> Added buildi…ng of IronPython to irtests.bat since it is needed for the language interop tests

Marshal.WriteObject was writing serialization data which includes  
instance variables, but the end of MarshalWriter.WriteAnObject was  
writing instance data as well. Removed writing of serialization data  
TimeOps.load had a bug causing local time to be loaded as utc time.  
Added building of IronPython to irtests.bat since it is needed for the  
language interop tests

Files changed:

- 

Merlin/External/Languages/IronRuby/mspec/ironruby-tags/core/marshal/dump\_tags.txt

- 

Merlin/External/Languages/IronRuby/mspec/rubyspec/core/marshal/dump\_spec.rb

- 

Merlin/External/Languages/IronRuby/mspec/rubyspec/core/marshal/fixtures/marshal\_data.rb

- 

Merlin/External/Languages/IronRuby/mspec/rubyspec/core/marshal/load\_spec.rb

- 

Merlin/External/Languages/IronRuby/mspec/rubyspec/core/time/\_load\_spec.rb

- 

Merlin/Main/Languages/Ruby/Libraries.LCA\_RESTRICTED/Builtins/Marshal.cs

- 

Merlin/Main/Languages/Ruby/Libraries.LCA\_RESTRICTED/Builtins/TimeOps.cs

- 

```
    Merlin/Main/Languages/Ruby/Scripts/irtests.bat

```

Thanks,  
Shri
