From mboxrd@z Thu Jan 1 00:00:00 1970 From: Detlev Zundel Date: Fri, 20 Mar 2009 11:45:45 +0100 Subject: [U-Boot] [PATCH v2] UBIFS: Missing offset relocation for compressor 'none' In-Reply-To: <200903200831.32171.sr@denx.de> (Stefan Roese's message of "Fri, 20 Mar 2009 08:31:32 +0100") References: <49C20B21.1080005@gmx.de> <200903200831.32171.sr@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Stefan, > On Thursday 19 March 2009, Michael Lawnick wrote: >> On systems where U-Boot is linked to another address than it really lays >> (e.g. backup image), calls via function pointers must be fixed with a >> '+= gd->reloc_off'. >> This was not done for none_compr in ubifs_compressors_init() what leads >> to system crash on ubifsmount command. >> >> Signed-off-by: Michael Lawnick > > Acked-by: Stefan Roese > > Please note that I explicitly didn't fold this patch into my latest UBIFS > patch series. We should apply it after UBIFS is available in mainline so that > Michael will be listed as author for this fix. I was not so sure about this, so I was waiting for your opinion, but are you sure that it is ok to remove the assignment? Doesn't other code depend on that? @@ -147,7 +147,10 @@ int __init ubifs_compressors_init(void) if (err) return err; - ubifs_compressors[UBIFS_COMPR_NONE] = &none_compr; + err = compr_init(&none_compr); + if (err) + return err; + return 0; } Cheers Detlev -- Narren sind alle, die es scheinen, und die Haelfte derer, die es nicht scheinen .. Jedoch ist der groesste Narr, wer es nicht zu sein glaubt und alle andern dafuer erklaert. --- Baltasar Gracian -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de